Fix warning when build with gcc7 #1297

Closed
wants to merge 4 commits into
from

Projects

None yet

3 participants

@alagoutte
Contributor

No description provided.

alagoutte added some commits Feb 28, 2017
@alagoutte alagoutte formdata: fix his statement may fall through [-Wimplicit-fallthrough=…
…] found by gcc7
b33182e
@alagoutte alagoutte ftp: fix his statement may fall through [-Wimplicit-fallthrough=] fou…
…nd by gcc7
1c0f860
@alagoutte alagoutte openssl (vtls): fix his statement may fall through [-Wimplicit-fallth…
…rough=] found by gcc7
03b4586
@alagoutte alagoutte tool_cb_dbg: fix his statement may fall through [-Wimplicit-fallthrou…
…gh=] found by gcc7
0060762

@alagoutte, thanks for your PR! By analyzing the history of the files in this pull request, we identified @bagder, @yangtse and @captain-caveman2k to be potential reviewers.

@jay jay added a commit that closed this pull request Mar 3, 2017
@alagoutte @jay alagoutte + jay build: fix gcc7 implicit fallthrough warnings
Mark intended fallthroughs with /* FALLTHROUGH */ so that gcc will know
it's expected and won't warn on [-Wimplicit-fallthrough=].

Closes #1297
cbff751
@jay jay closed this in cbff751 Mar 3, 2017
Owner
jay commented Mar 3, 2017

Thanks. I modified this slightly by moving them over since the ones already in the repo are done that way. So they look like this now:

case foo:
  bar;
  /* FALLTHROUGH */
case baz:
Contributor

Hi,

Ok for indent issue and merge in one commit

there is a another warning with gcc7, need to look before... (i think open a issue before)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment