Add fall through comment for missing break in case statement

droe / sslsplit

Transparent SSL/TLS interception

https://www.roe.ch/SSLsplit

BSD 2-Clause "Simplified" License

1.75k stars 328 forks source link

Add fall through comment for missing break in case statement #197

Closed sonertari closed 6 years ago

sonertari commented 6 years ago

Just a small improvement, as gcc (Ubuntu 7.3.0-16ubuntu3) 7.3.0 complains about missing breaks in case statements now:

privsep.c: In function ‘privsep_server_handle_req’:
privsep.c:314:10: warning: this statement may fall through [-Wimplicit-fallthrough=]
   mkpath = 1;
   ~~~~~~~^~~
privsep.c:315:2: note: here
  case PRIVSEP_REQ_OPENFILE: {
  ^~~~

sonertari commented 6 years ago

Sorry, this PR needs other commits be applied too. So I am closing it now.