[haiku-commits] Re: r36596 - haiku/trunk/src/apps/packageinstaller

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 04 May 2010 08:34:35 +0200

Stephan Aßmus <superstippi@xxxxxx> wrote:
> Am Montag, den 03.05.2010, 18:40 +0200 schrieb Jérôme Duval:
> > 2010/5/3  <superstippi@xxxxxx>:
> > > -                               if (volume.IsReadOnly() ||
> > > -                                               volume.
> > > GetRootDirectory(&mountPoint) != B_OK)
> > > +                               if (volume.IsReadOnly() || !
> > > volume.IsPersistent()
> > > +                                       || volume.
> > > GetRootDirectory(&mountPoint) != B_OK) {
> > >                                        continue;
> > > +                               }
> > Why adding braces ?
> Coding style requires it for multi-line if-statemens, which also goes
> for if-statements where just the if-clause itself makes it multi-
> line.

Actually, this is not the case. It's just something you and Ingo do
because you think it increases readability :-)

Bye,
   Axel.


Other related posts: