Author Don’t: I prefer short names so I’d rather
not change this. Unless you make
me? :)
Author Do: Best practice suggests omitting
obvious/generic terms. I’m not
sure how to reconcile that
advice with this request.
Reviewer Don’t: Why are you using this approach?
You’re adding unnecessary
complexity.
Reviewer Do: This concurrency model appears to
be adding complexity to the
system without any visible
performance benefit.
Reviewer Don’t: I don’t understand this.
Reviewer Do: If this is an optimization, can you
please add comments?
Author Don’t: That makes sense in some cases but
not here.
Author Do: I added a comment about why
it’s implemented that way.
