[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