Hi,

On Thu, Dec 08, 2016 at 05:04:53PM -0500, Selva Nair wrote:
> if (xxxx)
> {
>    ...
> }
> followed by
> else if (xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx) {
>  ....
> }
> 
> looks inconsitent and not pretty. 

Thanks, so I'm not alone with this :-)

> Why not just stick to:
> 
> if (carefree())
> {
>    follow_me();
> }
> else if (cautious())
> {
>    as_you_wish();
> }
> else
> {
>    abort();
> }
> 
> Looks easier to read.

This!

Maybe it's just my OCD speaking, but 

  if (something)
  {
      this();
  } else {
      that();
  }

really makes my skin crawl - I need the "i"s dotted, the "t"s crossed,
and braces aligned - vertically or horizontally, but not "diagonally".

So this would work for me, for "trivial conditionals" (= short condition,
non-crammed single-line statements):

  if (a>0)
     { do_this(); }
  else
     { do_that(); }

but since we agred on "braces get their own line", it's out as well.

gert

-- 
USENET is *not* the non-clickable part of WWW!
                                                           //www.muc.de/~gert/
Gert Doering - Munich, Germany                             g...@greenie.muc.de
fax: +49-89-35655025                        g...@net.informatik.tu-muenchen.de

Attachment: signature.asc
Description: PGP signature

------------------------------------------------------------------------------
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

Reply via email to