waxwing
· 1w
A comment by Luke Parker (guy who's the main author of FCMP stuff) struck me on a PR to introduce such a verification cache:
"We could just serialize key images, and fix that issue. I suggested in #8...
I should really show the follow up paragraph:
"While I'm not the King of Monero, I am frustrated my approval was given to a PR on a faulty premise. I'm trying to funnel this frustration into not having to worry about this anymore. Yes, serializing the entire TX is overkill. That's the point. It may be less efficient. This offers a 5x perf boost. I'd rather hear we left .1x on the table than hear we had a double spend in the next release."
I hope more developers of bitcoin security code don't feel bad about "overkill", either!