You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ma...@apache.org on 2017/10/11 07:50:42 UTC

[trafficserver] 09/25: Have Quic use make_unique

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

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

commit b17b572cfe62612168ec5c53b5673343fa0eec1c
Author: Bryan Call <bc...@apache.org>
AuthorDate: Tue Oct 3 18:05:06 2017 -0700

    Have Quic use make_unique
    
    (cherry picked from commit 36afa3a1708420b4232a63ab43a791e6fef83995)
---
 iocore/net/QUICNetVConnection.cc | 4 ++--
 iocore/net/quic/QUICTypes.h      | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 8801922..aff162a 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -412,7 +412,7 @@ QUICNetVConnection::state_handshake(int event, Event *data)
   case EVENT_IMMEDIATE: {
     // Start Implicit Shutdown. Because of no network activity for the duration of the idle timeout.
     this->remove_from_active_queue();
-    this->close({});
+    this->close(std::make_unique<QUICConnectionError>());
 
     // TODO: signal VC_EVENT_ACTIVE_TIMEOUT/VC_EVENT_INACTIVITY_TIMEOUT to application
     break;
@@ -472,7 +472,7 @@ QUICNetVConnection::state_connection_established(int event, Event *data)
   case EVENT_IMMEDIATE: {
     // Start Implicit Shutdown. Because of no network activity for the duration of the idle timeout.
     this->remove_from_active_queue();
-    this->close({});
+    this->close(std::make_unique<QUICConnectionError>());
 
     // TODO: signal VC_EVENT_ACTIVE_TIMEOUT/VC_EVENT_INACTIVITY_TIMEOUT to application
     break;
diff --git a/iocore/net/quic/QUICTypes.h b/iocore/net/quic/QUICTypes.h
index c2673d9..d25f36a 100644
--- a/iocore/net/quic/QUICTypes.h
+++ b/iocore/net/quic/QUICTypes.h
@@ -169,6 +169,7 @@ public:
 class QUICConnectionError : public QUICError
 {
 public:
+  QUICConnectionError() : QUICError() {}
   QUICConnectionError(const QUICErrorClass error_class, const QUICErrorCode error_code, const char *error_msg = nullptr)
     : QUICError(error_class, error_code, error_msg){};
 };

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