[trafficserver] 04/05: Fixed debug build on Fedora 26 with gcc7

[zw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit 78fb420fda1aef3dd5c9a8f3559da12b0928d45d
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon Jul 17 10:42:14 2017 +0200

    Fixed debug build on Fedora 26 with gcc7
    
    (cherry picked from commit c816a80483416c9d1a9bc0864a79b61db2c51ed1)
---
 iocore/net/P_UnixNet.h     | 2 ++
 iocore/net/Socks.cc        | 1 +
 proxy/http/HttpTransact.cc | 4 ++--
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/iocore/net/P_UnixNet.h b/iocore/net/P_UnixNet.h
index f514ce8..ab8fa3b 100644
--- a/iocore/net/P_UnixNet.h
+++ b/iocore/net/P_UnixNet.h
@@ -357,6 +357,7 @@ accept_error_seriousness(int res)
 #endif
     ink_assert(!"throttling misconfigured: set too high");
 #ifdef ENOBUFS
+  // fallthrough
   case -ENOBUFS:
 #endif
 #ifdef ENFILE
@@ -468,6 +469,7 @@ EventIO::close()
   switch (type) {
   default:
     ink_assert(!"case");
+  // fallthrough
   case EVENTIO_DNS_CONNECTION:
     return data.dnscon->close();
     break;
diff --git a/iocore/net/Socks.cc b/iocore/net/Socks.cc
index 331914a..4547140 100644
--- a/iocore/net/Socks.cc
+++ b/iocore/net/Socks.cc
@@ -119,6 +119,7 @@ SocksEntry::findServer()
 
   default:
     ink_assert(!"Unexpected event");
+  // fallthrough
   case PARENT_DIRECT:
   case PARENT_FAIL:
     memset(&server_addr, 0, sizeof(server_addr));
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index 9acb881..daf2e39 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -5170,7 +5170,7 @@ HttpTransact::get_ka_info_from_config(State *s, ConnectionAttributes *server_inf
   default:
     // The default is the "1" config, SEND_HTTP11_ALWAYS, but assert in debug builds since we shouldn't be here
     ink_assert(0);
-  // FALL THROUGH in a release build
+  // fallthrough
   case HttpConfigParams::SEND_HTTP11_ALWAYS:
     server_info->http_version = HTTPVersion(1, 1);
     break;
@@ -5213,7 +5213,7 @@ HttpTransact::get_ka_info_from_host_db(State *s, ConnectionAttributes *server_in
   default:
     // The default is the "1" config, SEND_HTTP11_ALWAYS, but assert in debug builds since we shouldn't be here
     ink_assert(0);
-  // FALL THROUGH in a release build
+  // fallthrough
   case HttpConfigParams::SEND_HTTP11_ALWAYS:
     force_http11 = true;
     break;

-- 
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.