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