[haiku-commits] Re: r33766 - haiku/trunk/src/kits/interface

  • From: Ingo Weinhold <ingo_weinhold@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 26 Oct 2009 08:52:13 +0800

On 2009-10-26 at 07:27:33 [+0800], Brecht Machiels <brecht@xxxxxxxxxxx> 
wrote:
> axeld@xxxxxxxxxxxxxxxx wrote:
> >      if (reply.FindData("key_buffer", B_ANY_TYPE, &key_array, 
> >      key_buffer_size)
> > -        != B_OK) {
> > +            != B_OK) {
> >          *map = 0; *key_buffer = 0;
> >          return;
> >      }
> 
> I've been wanting to indent an extra tab under an if to make it more
> clear that it it not part of the if body. The coding guidelines only
> ever show one tab however, which is what I stuck to.

I'm too lazy to reread the guidelines now, but the general idea is to add 
one indentation level per expression level. E.g.:

        if (foo
                && (bar
                        || foobar(with, long,
                                argument, list))) {
                ...
        }

> The /*! comment is not discussed in the coding guidelines.

Should better be remedied...

CU, Ingo

Other related posts: