On 2010-05-04 at 08:34:35 [+0200], Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote: > 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 :-) It does not explicitely mention this case, but the only example that comes close suggests the use of parenthesis, and I find it makes perfect sense. Why make rules more complicated than they need to be? Best regards, -Stephan