Re: [PATCH 1/1] scripts/checkpatch.pl: avoid false warning missing break

2017-09-10 Thread Joe Perches
On Sun, 2017-09-10 at 09:52 +0200, Heinrich Schuchardt wrote: > void foo(int a) > switch (a) { > case 'h': > fun1(); > exit(1); > default: > } > > creates a warning > Possible switch case/default not preceded by break > or fallthrough comment > >

Re: [PATCH 1/1] scripts/checkpatch.pl: avoid false warning missing break

2017-09-10 Thread Joe Perches
On Sun, 2017-09-10 at 09:52 +0200, Heinrich Schuchardt wrote: > void foo(int a) > switch (a) { > case 'h': > fun1(); > exit(1); > default: > } > > creates a warning > Possible switch case/default not preceded by break > or fallthrough comment > >

[PATCH 1/1] scripts/checkpatch.pl: avoid false warning missing break

2017-09-10 Thread Heinrich Schuchardt
void foo(int a) switch (a) { case 'h': fun1(); exit(1); default: } creates a warning Possible switch case/default not preceded by break or fallthrough comment exit( should be treated like return. Signed-off-by: Heinrich Schuchardt

[PATCH 1/1] scripts/checkpatch.pl: avoid false warning missing break

2017-09-10 Thread Heinrich Schuchardt
void foo(int a) switch (a) { case 'h': fun1(); exit(1); default: } creates a warning Possible switch case/default not preceded by break or fallthrough comment exit( should be treated like return. Signed-off-by: Heinrich Schuchardt ---