On Thu, Dec 8, 2016 at 4:50 PM, Steffan Karger <stef...@karger.me> wrote:
> .
>
> > This proposal allows for the "open bracing" question, which allows
> > '} else {' and '} else if () {'.
> >
> > So now the bike shed debate opens up ... Is this something we can get
> > consensus around?
>
> Even though I asked for this on the CodeStyle page, I have to agree with
> Gert's argument on IRC: we settled for Allman in Munich, so we should
> not re-open the bikeshed (even more ;) ). So, even though it's not my
> personal preference, I argue we should not allow this. Apologies for
> the noise...
>
I forgot to mention that, which was the only real comemnt I had:
if (xxxx)
{
...
}
followed by
else if (xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx) {
....
}
looks inconsitent and not pretty. Why not just stick to:
if (carefree())
{
follow_me();
}
else if (cautious())
{
as_you_wish();
}
else
{
abort();
}
Looks easier to read.
Selva
------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today.http://sdm.link/xeonphi
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
