Changes between Version 26 and Version 27 of PatchReviewProcess


Ignore:
Timestamp:
2013-09-20T20:04:29Z (11 years ago)
Author:
zooko
Comment:

English edit

Legend:

Unmodified
Added
Removed
Modified
  • PatchReviewProcess

    v26 v27  
    4343= Experiment: Suspicious Patch Review =
    4444
    45 Maybe it would be fun to imagine, while you're reviewing a patch, that the author has been compelled to slip a Trojan into the patch by evil adversaries, and it is your job to notice it! That's why you are motivated to request that the author rewrite or explain any confusing parts, because any parts of the patch that confuse the code-reviewer are parts where a Trojan could slip through. Of course, well-written Trojans are indistinguishable from normal old innocent bugs, so if you find a bug this way, give the author the benefit of the doubt and assume that it was an honest mistake.
     45Maybe it would be fun to imagine, while you're reviewing a patch, that the author has been compelled by evil adversaries to slip a Trojan into the patch, and it is your job to notice it! That's why you are motivated to request that the author rewrite or explain any confusing parts, because any parts of the patch that confuse the code-reviewer are parts where a Trojan could slip through. Of course, well-written Trojans are indistinguishable from normal old innocent bugs, so if you find a bug this way, give the author the benefit of the doubt and assume that it was an honest mistake.
    4646
    4747== Using trac and github ==