Re: [PATCH app/xdm] Add missing brackets to multi-statement if in SaveServerAuthorizations

2018-05-06 Thread Julien Cristau
On Sat, May 5, 2018 at 12:33:12 -0700, Alan Coopersmith wrote: > Introduced by commit 5222d28e8d8e5b4cc > > Reported by gcc 7.3: > > auth.c: In function ‘SaveServerAuthorizations’: > auth.c:447:6: warning: this ‘if’ clause does not guard... > [-Wmisleading-indentation] > if

Re: [PATCH app/xdm] Add missing brackets to multi-statement if in SaveServerAuthorizations

2018-05-06 Thread Matthieu Herrb
On Sat, May 05, 2018 at 12:33:12PM -0700, Alan Coopersmith wrote: > Introduced by commit 5222d28e8d8e5b4cc > > Reported by gcc 7.3: > > auth.c: In function ‘SaveServerAuthorizations’: > auth.c:447:6: warning: this ‘if’ clause does not guard... > [-Wmisleading-indentation] > if

[PATCH app/xdm] Add missing brackets to multi-statement if in SaveServerAuthorizations

2018-05-05 Thread Alan Coopersmith
Introduced by commit 5222d28e8d8e5b4cc Reported by gcc 7.3: auth.c: In function ‘SaveServerAuthorizations’: auth.c:447:6: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] if (auths[i]->data_length > 0) ^~ auth.c:452:3: note: ...this statement, but the latter is