Jason Dillon wrote:
So far 2+ days, several patches... one PMC +1, one non-PMC +1 (with caveat to ping JVZ)... now crazy problems with diff/patch.. which I'm not exactly sure how that affects the current votes... or does adding a new version of the patch negate anything else voted upon.
IMO, a vote for a patch would be need to be restarted if the changes between the previous patch and the newer version of the patch are not trivial. Trival meaning:

* documentation changes
* non-controversial non-semantic style changes such as fixing indentation and adding comments. Trivial changes do not include code changes or changes that affect the operation of the build.

To make it easier for reviewers when a new version of a patch is generated, it would be worthwhile adding a comment on what has changed since the previous patch.

Do the above patch vote negation guidelines sound reasonable to everyone?

Thanks,

John

Reply via email to