You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bc...@apache.org on 2017/07/17 16:12:49 UTC
[trafficserver] branch master updated: Fixed debug build on Fedora
26 with gcc7
This is an automated email from the ASF dual-hosted git repository.
bcall pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new c816a80 Fixed debug build on Fedora 26 with gcc7
c816a80 is described below
commit c816a80483416c9d1a9bc0864a79b61db2c51ed1
Author: Bryan Call <bc...@apache.org>
AuthorDate: Mon Jul 17 10:42:14 2017 +0200
Fixed debug build on Fedora 26 with gcc7
---
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 97a1aea..b570b91 100644
--- a/iocore/net/P_UnixNet.h
+++ b/iocore/net/P_UnixNet.h
@@ -353,6 +353,7 @@ accept_error_seriousness(int res)
#endif
ink_assert(!"throttling misconfigured: set too high");
#ifdef ENOBUFS
+ // fallthrough
case -ENOBUFS:
#endif
#ifdef ENFILE
@@ -446,6 +447,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 568ae05..ca6c876 100644
--- a/iocore/net/Socks.cc
+++ b/iocore/net/Socks.cc
@@ -121,6 +121,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 711f6fd..ce7b672 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -5005,7 +5005,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;
@@ -5048,7 +5048,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>'].