Most of my pull requests to Bitcoin Core tend to be about 10 lines of code, though between my initial PR and the final merged commit, almost every line tend to get rewritten as a result of feedback from reviewers. 😬

Reply to this note

Please Login to reply.

Discussion

This is the way

Sounds like almost all security sensitive code

Is the merged and changed code usually an improvement?

Sassy.

Oh now I see what you mean. I thought you were doubting his ability to improve Bitcoin core 😂

Indeed, I don't think I've ever rejected a reviewer suggestion as being a bad idea.