[PATCH] CodingStyle: add an extra line for the braces explaination

Thiago Macieira thiago at macieira.org
Wed Mar 25 17:16:10 PDT 2015


On Monday 23 March 2015 16:03:28 Lubomir I. Ivanov wrote:
> > 18 March 2015 at 02:22, Lubomir I. Ivanov <neolit123 at gmail.com> wrote:
> > From: "Lubomir I. Ivanov" <neolit123 at gmail.com>
> > 
> > The extra call "dosomethingelse()" is added with the
> > intention that user will not be confused to add braces
> > around single lined "if" conditions.
> > 
> > Signed-off-by: Lubomir I. Ivanov <neolit123 at gmail.com>
> > ---
> > 
> >  CodingStyle | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/CodingStyle b/CodingStyle
> > index 75aa1d4..ead24d0 100644
> > --- a/CodingStyle
> > +++ b/CodingStyle
> > @@ -42,6 +42,7 @@ Basic rules
> > 
> >         if (condition) {
> >         
> >                 dosomething();
> > 
> > +               dosomethingelse();
> > 
> >         }
> >  
> >  - both sides of an if / else clause either use or do not use curly 
braces:
> bump.

Makes sense to me.
-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
   Software Architect - Intel Open Source Technology Center
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358



More information about the subsurface mailing list