From 15ed9f87e3a1f91bd7b4a5291edf268b843e5987 Mon Sep 17 00:00:00 2001 From: Marcel Raad Date: Mon, 9 Jul 2018 18:52:05 +0200 Subject: curl-compilers: enable -Wimplicit-fallthrough=4 for GCC This enables level 4 instead of the default level 3, which of the currently used comments only allows /* FALLTHROUGH */ to silence the warning. Closes https://github.com/curl/curl/pull/2747 --- lib/formdata.c | 3 ++- lib/http_ntlm.c | 2 +- lib/ssh.c | 4 ++-- lib/telnet.c | 2 +- lib/transfer.c | 4 ++-- 5 files changed, 8 insertions(+), 7 deletions(-) (limited to 'lib') diff --git a/lib/formdata.c b/lib/formdata.c index 450a0e5d8..202d930c7 100644 --- a/lib/formdata.c +++ b/lib/formdata.c @@ -302,7 +302,8 @@ CURLFORMcode FormAdd(struct curl_httppost **httppost, * Set the contents property. */ case CURLFORM_PTRCONTENTS: - current_form->flags |= HTTPPOST_PTRCONTENTS; /* fall through */ + current_form->flags |= HTTPPOST_PTRCONTENTS; + /* FALLTHROUGH */ case CURLFORM_COPYCONTENTS: if(current_form->value) return_value = CURL_FORMADD_OPTION_TWICE; diff --git a/lib/http_ntlm.c b/lib/http_ntlm.c index fd5540b5d..a9b33f98e 100644 --- a/lib/http_ntlm.c +++ b/lib/http_ntlm.c @@ -228,7 +228,7 @@ CURLcode Curl_output_ntlm(struct connectdata *conn, bool proxy) /* connection is already authenticated, * don't send a header in future requests */ ntlm->state = NTLMSTATE_LAST; - /* fall-through */ + /* FALLTHROUGH */ case NTLMSTATE_LAST: Curl_safefree(*allocuserpwd); authp->done = TRUE; diff --git a/lib/ssh.c b/lib/ssh.c index 98153040b..5d81f649c 100644 --- a/lib/ssh.c +++ b/lib/ssh.c @@ -659,7 +659,7 @@ static CURLcode ssh_statemach_act(struct connectdata *conn, bool *block) libssh2_session_set_blocking(sshc->ssh_session, 0); state(conn, SSH_S_STARTUP); - /* fall-through */ + /* FALLTHROUGH */ case SSH_S_STARTUP: rc = libssh2_session_startup(sshc->ssh_session, (int)sock); @@ -675,7 +675,7 @@ static CURLcode ssh_statemach_act(struct connectdata *conn, bool *block) state(conn, SSH_HOSTKEY); - /* fall-through */ + /* FALLTHROUGH */ case SSH_HOSTKEY: /* * Before we authenticate we should check the hostkey's fingerprint diff --git a/lib/telnet.c b/lib/telnet.c index 031d61f4e..05fe744db 100644 --- a/lib/telnet.c +++ b/lib/telnet.c @@ -1606,7 +1606,7 @@ static CURLcode telnet_do(struct connectdata *conn, bool *done) case 0: /* timeout */ pfd[0].revents = 0; pfd[1].revents = 0; - /* fall through */ + /* FALLTHROUGH */ default: /* read! */ if(pfd[0].revents & POLLIN) { /* read data from network */ diff --git a/lib/transfer.c b/lib/transfer.c index ab9094adc..298208703 100644 --- a/lib/transfer.c +++ b/lib/transfer.c @@ -1519,7 +1519,7 @@ static size_t strlen_url(const char *url, bool relative) switch(*ptr) { case '?': left = FALSE; - /* fall through */ + /* FALLTHROUGH */ default: if(urlchar_needs_escaping(*ptr)) newlen += 2; @@ -1564,7 +1564,7 @@ static void strcpy_url(char *output, const char *url, bool relative) switch(*iptr) { case '?': left = FALSE; - /* fall through */ + /* FALLTHROUGH */ default: if(urlchar_needs_escaping(*iptr)) { snprintf(optr, 4, "%%%02x", *iptr); -- cgit v1.2.3