[trafficserver] branch quic-latest updated: Fix warnings on compile with gcc

[maskit]

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

maskit pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/quic-latest by this push:
     new aea566d  Fix warnings on compile with gcc
aea566d is described below

commit aea566de628622436347fbcf7080b0585ab7e7a9
Author: Masakazu Kitajo <mas...@apache.org>
AuthorDate: Wed Mar 7 15:42:52 2018 +0900

    Fix warnings on compile with gcc
---
 iocore/net/QUICNet.cc     |  2 +-
 proxy/hq/QUICSimpleApp.cc | 12 ++++--------
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/iocore/net/QUICNet.cc b/iocore/net/QUICNet.cc
index e153c35..5bc9761 100644
--- a/iocore/net/QUICNet.cc
+++ b/iocore/net/QUICNet.cc
@@ -77,8 +77,8 @@ QUICPollCont::_process_long_header_packet(QUICPollEvent *e, 
NetHandler *nh)
       vc->read.triggered = 1;
       vc->handle_received_packet(p);
       vc->handleEvent(QUIC_EVENT_PACKET_READ_READY, nullptr);
-      return;
     }
+    return;
   case QUICPacketType::ZERO_RTT_PROTECTED:
   // TODO:: do something ?
   // break;
diff --git a/proxy/hq/QUICSimpleApp.cc b/proxy/hq/QUICSimpleApp.cc
index 054075d..36fd7d7 100644
--- a/proxy/hq/QUICSimpleApp.cc
+++ b/proxy/hq/QUICSimpleApp.cc
@@ -70,7 +70,7 @@ QUICSimpleApp::main_event_handler(int event, Event *data)
 
   switch (event) {
   case VC_EVENT_READ_READY:
-  case VC_EVENT_READ_COMPLETE: {
+  case VC_EVENT_READ_COMPLETE:
     if (stream_io->is_read_avail_more_than(0)) {
       if (txn == nullptr) {
         txn = new HQClientTransaction(this->_client_session, stream_io);
@@ -80,24 +80,20 @@ QUICSimpleApp::main_event_handler(int event, Event *data)
       } else {
         txn->handleEvent(event);
       }
-      break;
     }
-  }
+    break;
   case VC_EVENT_WRITE_READY:
-  case VC_EVENT_WRITE_COMPLETE: {
+  case VC_EVENT_WRITE_COMPLETE:
     if (txn != nullptr) {
       txn->handleEvent(event);
     }
-
     break;
-  }
   case VC_EVENT_EOS:
   case VC_EVENT_ERROR:
   case VC_EVENT_INACTIVITY_TIMEOUT:
-  case VC_EVENT_ACTIVE_TIMEOUT: {
+  case VC_EVENT_ACTIVE_TIMEOUT:
     ink_assert(false);
     break;
-  }
   default:
     break;
   }

-- 
To stop receiving notification emails like this one, please contact
mas...@apache.org.