Ramkumar Ramachandra wrote: > - "revert: report fine-grained error messages from insn parser" arises > from Jonathan's request to split "revert: allow mixed pick and > revert instructions". Just to be clear: I wasn't directly requesting that you do anything. If I were, then you could carefully read my requirements, fulfill them, and you would be done. Instead, I was reviewing the patch and giving my reaction. After receiving that information, one has plenty of choices: - add documentation to avoid the confusion the reaction was based on - rework to fix the underlying problem that caused the reaction - think carefully about it, conclude that the reviewer is crazy, and ignore it - drop the patch - send out a call for help to get others to help work on the underlying problem - ask for clarification ... >From my point of view as a reviewer, I am happiest when someone figures out how I missed the point and comes up with some fix that addresses the underlying problem instead (and, incidentally, gets rid of the symptom that provoked my reaction on the way). Well, you get the idea. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html