This happens so frequently to me. I'll Spend an hour running benchmarks and then post a PR. Someone chimes in and says "method x is faster" with nothing to back it up. I have to benchmark again, and then my way is still faster.
Do you have to benchmark again? Why not just say something like "post evidence, please. You're free to use my benchmark source code"? (I guess it's a variant of the somewhat annoying [citation needed], but sometimes it is apt to post something like that.)
I've learned to do this, but for me it wasn't obvious at first. At the end of the day if it's my PR, it's my job to provide the supporting evidence as to why that code should be merged and not some other code. If you respond "no you benchmark it" and they never respond, the ticket can sometimes get stuck in limbo.
lomnakkus|11 years ago
thinkbohemian|11 years ago