On 2 November 2016 at 14:46, Aric Stewart aric@codeweavers.com wrote:
Here is a discussion I would love to have, especially with the greater dependence on non Alexandre reviewers
What are wine's coding style requirements? How much do style error block review/sign-off? Can a reviewer just change style, how much changing requires a re-sign-off from the original author?
It's largely at the discretion of the module's maintainer, and I think that's how it should be. Which is to say, I'll happily discuss my personal answers/style preferences with you, and I could probably be convinced to write them down on the wiki, but that won't necessarily apply to any other maintainers. If on the other hand the argument is "Wine should have a single unified coding style", I just don't see that happening.
I can't help but wonder if the issue you're attempting to address is really that different people have different standards, or if there's a different, perhaps more tractable, issue you'd like to address? I.e., it may be more helpful to start with describing the specific problem than with an attempted solution.
I am noticing that our coding guidelines block on the wiki is, small, incomplete and I personally have been told to do things directly in opposition to some of the points on this page:
I'd be curious about specific examples, but those are guidelines and not hard rules.