On Mon, 2008-07-21 at 12:19 +0300, Vesa-Matti J Kari wrote: > Hello, > > I'm planning to do more trivial fixes to the SELinux kernel subsystem to > make the coding style look as unified as possible. Before continuing, > however, I would like to ask for an opinion, because I do not want to > waste my time. > > There currently exist mixed styles such as: > > while (cur != NULL) > > vs > > while (cur) > > and of course the related "if"- and "for"-variants. > > My questions: > > 1) Is there a general agreement that only one style should be used? I'd agree with that, yes. > 2) If answer to the question 1 is "yes", which one is the preferred style? I'm inclined to the latter as it is more concise and loses nothing in semantics, but I'm open to hearing the other side of it. A note however: the core selinux developers are likely to be somewhat unresponsive to non-critical patches right now because we are at the SELinux mini-summit and Linux Symposium this week. > > Thanks for the information. > > Best regards, > vmk > -- > ************************************************************************ > Tietotekniikkaosasto / Helsingin yliopisto > IT Department / University of Helsinki > ************************************************************************ > -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.