macdrv: implement getting and setting the screen saver state. if(Style)

Hi,

Ken Thomases wrote:
> > +                    if(success != kIOReturnSuccess)
> > Another style nitpick: please put a space between "if" and the condition.
> >  That applies to the "if(count2)" above, too.

I find this over the top.  Where is the rule?
Andrew Eikum's mmdevapi code -- which I don't consider very old in Wine --
uses exactly 0 space between if and ().

So what is it that the Wine "project" can tell new contributors?

The only rule I see is: "Look at the code surrounding the patch" and
its corollary: "if you create a new module, you're the first and define
the style others will have to follow."

Regards,
	Jörg Höhle

[Christian Costa]

2013/1/22  <Joerg-Cyril.Hoehle(a)t-systems.com>:
> Hi,
>
> Ken Thomases wrote:
> > > +                    if(success != kIOReturnSuccess)
> > > Another style nitpick: please put a space between "if" and the condition.
> > >  That applies to the "if(count2)" above, too.
>
> I find this over the top.  Where is the rule?
> Andrew Eikum's mmdevapi code -- which I don't consider very old in Wine --
> uses exactly 0 space between if and ().
>
> So what is it that the Wine "project" can tell new contributors?
>
> The only rule I see is: "Look at the code surrounding the patch" and
> its corollary: "if you create a new module, you're the first and define
> the style others will have to follow."

Hi,

Just some stats regard "if" before Dan beat me at it ;) :
- without space => 39979
- with space => 144133

The space is better to make the distinction between C keyword and
function name. Off course it's a personal taste. :)

Bye,
Christian

[Ken Thomases]

On Jan 22, 2013, at 6:38 AM, <Joerg-Cyril.Hoehle(a)t-systems.com> <Joerg-Cyril.Hoehle(a)t-systems.com> wrote:
> Ken Thomases wrote:
> > > +                    if(success != kIOReturnSuccess)
> > > Another style nitpick: please put a space between "if" and the condition.
> > > That applies to the "if(count2)" above, too.
>
> I find this over the top.

Well, I phrased it as a style nitpick (a.k.a. a matter of taste and a minor issue).  I also said "please".  I don't think it was too over the top.
> Where is the rule?
> … The only rule I see is: "Look at the code surrounding the patch" and
> its corollary: "if you create a new module, you're the first and define
> the style others will have to follow."

Well, there's also the corollary "be self-consistent within a single patch" which applied in this case.  Those guidelines (I wouldn't call them "rules") seem sane and fairly self-evident.  In any case, they are already documented here:
http://wiki.winehq.org/SubmittingPatches#head-22ead28485239d5028d63061c7d38f...

I have no power to enforce anything.  Alexandre may or may not consider my patch reviews, or those of anybody else, while deciding how to treat the patches in his incoming queue.  The reviews are simply a joint effort of all Wine developers to help keep quality up and take some of the burden off of Alexandre.

If the code submitter chooses to ignore my suggestions, they can.  I probably won't keep harping on the same minor style issue for future versions.

-Ken