trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bc...@apache.org
Subject [trafficserver] branch master updated: Ran clang-tidy to add C++11 override to methods
Date Thu, 13 Apr 2017 21:27:59 GMT
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  87df80a   Ran clang-tidy to add C++11 override to methods
87df80a is described below

commit 87df80a10335cc7bf6c78b98109730580740ca9a
Author: Bryan Call <bcall@apache.org>
AuthorDate: Thu Apr 6 11:31:42 2017 -0700

    Ran clang-tidy to add C++11 override to methods
---
 example/cppapi/async_http_fetch/AsyncHttpFetch.cc  | 22 +++---
 .../AsyncHttpFetchStreaming.cc                     | 10 +--
 example/cppapi/async_timer/AsyncTimer.cc           |  4 +-
 example/cppapi/boom/boom.cc                        |  4 +-
 example/cppapi/clientredirect/ClientRedirect.cc    |  6 +-
 example/cppapi/clientrequest/ClientRequest.cc      |  6 +-
 .../CustomErrorRemapPlugin.cc                      |  2 +-
 example/cppapi/customresponse/CustomResponse.cc    |  6 +-
 example/cppapi/globalhook/GlobalHookPlugin.cc      |  4 +-
 .../GzipTransformationPlugin.cc                    | 20 +++---
 example/cppapi/intercept/intercept.cc              |  8 +--
 .../InternalTransactionHandling.cc                 | 10 +--
 example/cppapi/logger_example/LoggerExample.cc     |  4 +-
 .../MultipleTransactionHookPlugins.cc              | 14 ++--
 .../NullTransformationPlugin.cc                    | 18 ++---
 example/cppapi/post_buffer/PostBuffer.cc           | 10 +--
 example/cppapi/remap_plugin/RemapPlugin.cc         |  2 +-
 example/cppapi/serverresponse/ServerResponse.cc    |  6 +-
 example/cppapi/stat_example/StatExample.cc         |  4 +-
 .../cppapi/timeout_example/TimeoutExamplePlugin.cc |  8 +--
 .../transactionhook/TransactionHookPlugin.cc       |  8 +--
 iocore/cache/CachePages.cc                         |  2 +-
 iocore/cache/CachePagesInternal.cc                 |  2 +-
 iocore/cache/RamCacheCLFUS.cc                      | 10 +--
 iocore/cache/RamCacheLRU.cc                        | 10 +--
 iocore/cluster/ClusterAPI.cc                       |  4 +-
 iocore/hostdb/HostDB.cc                            |  2 +-
 iocore/hostdb/test_RefCountCache.cc                |  2 +-
 iocore/net/UnixNetProcessor.cc                     |  2 +-
 iocore/net/UnixUDPNet.cc                           |  2 +-
 lib/records/RecProcess.cc                          |  2 +-
 lib/ts/IpMap.cc                                    | 20 +++---
 lib/ts/test_Ptr.cc                                 |  2 +-
 mgmt/ProxyConfig.cc                                |  2 +-
 plugins/experimental/balancer/hash.cc              |  4 +-
 plugins/experimental/balancer/roundrobin.cc        |  4 +-
 .../experimental/cache_promote/cache_promote.cc    | 16 ++---
 .../experimental/stream_editor/stream_editor.cc    | 32 ++++-----
 .../experimental/webp_transform/ImageTransform.cc  | 12 ++--
 proxy/ControlBase.cc                               | 84 +++++++++++-----------
 proxy/Main.cc                                      |  4 +-
 proxy/PluginVC.cc                                  |  2 +-
 proxy/RegressionSM.cc                              |  8 +--
 proxy/SocksProxy.cc                                |  2 +-
 proxy/shared/UglyLogStubs.cc                       |  4 +-
 45 files changed, 205 insertions(+), 205 deletions(-)

diff --git a/example/cppapi/async_http_fetch/AsyncHttpFetch.cc b/example/cppapi/async_http_fetch/AsyncHttpFetch.cc
index 3901add..7ffb5c6 100644
--- a/example/cppapi/async_http_fetch/AsyncHttpFetch.cc
+++ b/example/cppapi/async_http_fetch/AsyncHttpFetch.cc
@@ -56,13 +56,13 @@ public:
   DelayedAsyncHttpFetch(string request, HttpMethod method, std::shared_ptr<Mutex> mutex)
     : AsyncHttpFetch(request, method), mutex_(mutex), timer_(nullptr){};
   void
-  run()
+  run() override
   {
     timer_ = new AsyncTimer(AsyncTimer::TYPE_ONE_OFF, 1000 /* 1s */);
     Async::execute(this, timer_, mutex_);
   }
   void
-  handleAsyncComplete(AsyncTimer & /*timer ATS_UNUSED */)
+  handleAsyncComplete(AsyncTimer & /*timer ATS_UNUSED */) override
   {
     TS_DEBUG(TAG, "Receiver should not be reachable");
     assert(!getDispatchController()->dispatch());
@@ -73,7 +73,7 @@ public:
   {
     return getDispatchController()->isEnabled();
   }
-  ~DelayedAsyncHttpFetch() { delete timer_; }
+  ~DelayedAsyncHttpFetch() override { delete timer_; }
 
 private:
   std::shared_ptr<Mutex> mutex_;
@@ -95,7 +95,7 @@ public:
   }
 
   void
-  handleSendRequestHeaders(Transaction & /*transaction ATS_UNUSED */)
+  handleSendRequestHeaders(Transaction & /*transaction ATS_UNUSED */) override
   {
     Async::execute<AsyncHttpFetch>(this, new AsyncHttpFetch("http://127.0.0.1/"), getMutex());
     ++num_fetches_pending_;
@@ -123,7 +123,7 @@ public:
   }
 
   void
-  handleAsyncComplete(AsyncHttpFetch &async_http_fetch)
+  handleAsyncComplete(AsyncHttpFetch &async_http_fetch) override
   {
     // This will be called when our async event is complete.
     TS_DEBUG(TAG, "AsyncHttpFetch completed");
@@ -131,14 +131,14 @@ public:
   }
 
   void
-  handleAsyncComplete(AsyncHttpFetch2 &async_http_fetch)
+  handleAsyncComplete(AsyncHttpFetch2 &async_http_fetch) override
   {
     // This will be called when our async event is complete.
     TS_DEBUG(TAG, "AsyncHttpFetch2 completed");
     handleAnyAsyncComplete(async_http_fetch);
   }
 
-  virtual ~TransactionHookPlugin()
+  ~TransactionHookPlugin() override
   {
     TS_DEBUG(TAG, "Destroyed TransactionHookPlugin!");
     // since we die right away, we should not receive the callback for this (using POST request this time)
@@ -146,13 +146,13 @@ public:
   }
 
   void
-  handleAsyncComplete(AsyncHttpFetch3 & /* async_http_fetch ATS_UNUSED */)
+  handleAsyncComplete(AsyncHttpFetch3 & /* async_http_fetch ATS_UNUSED */) override
   {
     assert(!"AsyncHttpFetch3 shouldn't have completed!");
   }
 
   void
-  handleAsyncComplete(DelayedAsyncHttpFetch & /*async_http_fetch ATS_UNUSED */)
+  handleAsyncComplete(DelayedAsyncHttpFetch & /*async_http_fetch ATS_UNUSED */) override
   {
     assert(!"Should've been canceled!");
   }
@@ -198,8 +198,8 @@ public:
     registerHook(HOOK_READ_REQUEST_HEADERS_POST_REMAP);
   }
 
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Received a request in handleReadRequestHeadersPostRemap.");
 
diff --git a/example/cppapi/async_http_fetch_streaming/AsyncHttpFetchStreaming.cc b/example/cppapi/async_http_fetch_streaming/AsyncHttpFetchStreaming.cc
index dcb78e2..e91a310 100644
--- a/example/cppapi/async_http_fetch_streaming/AsyncHttpFetchStreaming.cc
+++ b/example/cppapi/async_http_fetch_streaming/AsyncHttpFetchStreaming.cc
@@ -47,10 +47,10 @@ public:
   {
     main_url_ = transaction.getClientRequest().getUrl().getUrlString();
   }
-  void consume(const string &data, InterceptPlugin::RequestDataType type);
-  void handleInputComplete();
-  void handleAsyncComplete(AsyncHttpFetch &async_http_fetch);
-  ~Intercept();
+  void consume(const string &data, InterceptPlugin::RequestDataType type) override;
+  void handleInputComplete() override;
+  void handleAsyncComplete(AsyncHttpFetch &async_http_fetch) override;
+  ~Intercept() override;
 
 private:
   Transaction &transaction_;
@@ -68,7 +68,7 @@ public:
     GlobalPlugin::registerHook(Plugin::HOOK_READ_REQUEST_HEADERS_PRE_REMAP);
   }
   void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     transaction.addPlugin(new Intercept(transaction));
     TS_DEBUG(TAG, "Added intercept");
diff --git a/example/cppapi/async_timer/AsyncTimer.cc b/example/cppapi/async_timer/AsyncTimer.cc
index b0d5e90..b08bbee 100644
--- a/example/cppapi/async_timer/AsyncTimer.cc
+++ b/example/cppapi/async_timer/AsyncTimer.cc
@@ -38,7 +38,7 @@ public:
   }
 
   void
-  handleAsyncComplete(AsyncTimer &timer ATSCPPAPI_UNUSED)
+  handleAsyncComplete(AsyncTimer &timer ATSCPPAPI_UNUSED) override
   {
     TS_DEBUG(TAG, "Got timer event in object %p!", this);
     if ((type_ == AsyncTimer::TYPE_ONE_OFF) || (max_instances_ && (++instance_count_ == max_instances_))) {
@@ -47,7 +47,7 @@ public:
     }
   }
 
-  ~TimerEventReceiver() { delete timer_; }
+  ~TimerEventReceiver() override { delete timer_; }
 
 private:
   int max_instances_;
diff --git a/example/cppapi/boom/boom.cc b/example/cppapi/boom/boom.cc
index dfe14d8..ca523a7 100644
--- a/example/cppapi/boom/boom.cc
+++ b/example/cppapi/boom/boom.cc
@@ -312,7 +312,7 @@ public:
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     transaction.getClientResponse().setStatusCode(status_);
     transaction.getClientResponse().setReasonPhrase(reason_);
@@ -376,7 +376,7 @@ public:
   }
 
   // Upcall method that is called for every transaction.
-  void handleReadResponseHeaders(Transaction &transaction);
+  void handleReadResponseHeaders(Transaction &transaction) override;
 
 private:
   BoomGlobalPlugin();
diff --git a/example/cppapi/clientredirect/ClientRedirect.cc b/example/cppapi/clientredirect/ClientRedirect.cc
index a6b46ea..051d8f6 100644
--- a/example/cppapi/clientredirect/ClientRedirect.cc
+++ b/example/cppapi/clientredirect/ClientRedirect.cc
@@ -48,7 +48,7 @@ public:
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     transaction.getClientResponse().setStatusCode(HTTP_STATUS_MOVED_TEMPORARILY);
     transaction.getClientResponse().setReasonPhrase("Moved Temporarily");
@@ -56,7 +56,7 @@ public:
     transaction.resume();
   }
 
-  virtual ~ClientRedirectTransactionPlugin() {}
+  ~ClientRedirectTransactionPlugin() override {}
 
 private:
   string location_;
@@ -67,7 +67,7 @@ class ClientRedirectGlobalPlugin : public GlobalPlugin
 public:
   ClientRedirectGlobalPlugin() { registerHook(HOOK_SEND_REQUEST_HEADERS); }
   void
-  handleSendRequestHeaders(Transaction &transaction)
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     if (transaction.getClientRequest().getUrl().getQuery().find("redirect=1") != string::npos) {
       transaction.addPlugin(new ClientRedirectTransactionPlugin(transaction, "http://www.linkedin.com/"));
diff --git a/example/cppapi/clientrequest/ClientRequest.cc b/example/cppapi/clientrequest/ClientRequest.cc
index eb96c49..3179813 100644
--- a/example/cppapi/clientrequest/ClientRequest.cc
+++ b/example/cppapi/clientrequest/ClientRequest.cc
@@ -44,7 +44,7 @@ public:
   }
 
   void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     cout << "Hello from handleReadRequesHeadersPreRemap!" << endl;
 
@@ -69,7 +69,7 @@ public:
   }
 
   void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     cout << "Hello from handleReadRequesHeadersPostRemap!" << endl;
 
@@ -123,7 +123,7 @@ public:
   }
 
   void
-  handleSendRequestHeaders(Transaction &transaction)
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     cout << "Hello from handleSendRequestHeaders!" << endl;
     cout << "---------------------IP INFORMATION-----------------" << endl;
diff --git a/example/cppapi/custom_error_remap_plugin/CustomErrorRemapPlugin.cc b/example/cppapi/custom_error_remap_plugin/CustomErrorRemapPlugin.cc
index a3e599f..dea2fab 100644
--- a/example/cppapi/custom_error_remap_plugin/CustomErrorRemapPlugin.cc
+++ b/example/cppapi/custom_error_remap_plugin/CustomErrorRemapPlugin.cc
@@ -33,7 +33,7 @@ class MyRemapPlugin : public RemapPlugin
 public:
   MyRemapPlugin(void **instance_handle) : RemapPlugin(instance_handle) {}
   Result
-  doRemap(const Url &map_from_url, const Url &map_to_url, Transaction &transaction, bool &redirect)
+  doRemap(const Url &map_from_url, const Url &map_to_url, Transaction &transaction, bool &redirect) override
   {
     if (transaction.getClientRequest().getUrl().getQuery().find("custom=1") != string::npos) {
       transaction.setStatusCode(HTTP_STATUS_FORBIDDEN);
diff --git a/example/cppapi/customresponse/CustomResponse.cc b/example/cppapi/customresponse/CustomResponse.cc
index 2d4de5f..69ed4a9 100644
--- a/example/cppapi/customresponse/CustomResponse.cc
+++ b/example/cppapi/customresponse/CustomResponse.cc
@@ -53,14 +53,14 @@ public:
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     transaction.getClientResponse().setStatusCode(status_);
     transaction.getClientResponse().setReasonPhrase(reason_);
     transaction.resume();
   }
 
-  virtual ~CustomResponseTransactionPlugin() {}
+  ~CustomResponseTransactionPlugin() override {}
 
 private:
   HttpStatus status_;
@@ -73,7 +73,7 @@ class ClientRedirectGlobalPlugin : public GlobalPlugin
 public:
   ClientRedirectGlobalPlugin() { registerHook(HOOK_SEND_REQUEST_HEADERS); }
   void
-  handleSendRequestHeaders(Transaction &transaction)
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     if (transaction.getClientRequest().getUrl().getQuery().find("custom=1") != string::npos) {
       transaction.addPlugin(new CustomResponseTransactionPlugin(transaction, HTTP_STATUS_OK, "Ok",
diff --git a/example/cppapi/globalhook/GlobalHookPlugin.cc b/example/cppapi/globalhook/GlobalHookPlugin.cc
index a94e348..cccf900 100644
--- a/example/cppapi/globalhook/GlobalHookPlugin.cc
+++ b/example/cppapi/globalhook/GlobalHookPlugin.cc
@@ -33,8 +33,8 @@ class GlobalHookPlugin : public GlobalPlugin
 {
 public:
   GlobalHookPlugin() { registerHook(HOOK_READ_REQUEST_HEADERS_PRE_REMAP); }
-  virtual void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     std::cout << "Hello from handleReadRequesHeadersPreRemap!" << std::endl;
     transaction.resume();
diff --git a/example/cppapi/gzip_transformation/GzipTransformationPlugin.cc b/example/cppapi/gzip_transformation/GzipTransformationPlugin.cc
index 35968ef..b4babb5 100644
--- a/example/cppapi/gzip_transformation/GzipTransformationPlugin.cc
+++ b/example/cppapi/gzip_transformation/GzipTransformationPlugin.cc
@@ -95,7 +95,7 @@ public:
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Added X-Content-Transformed header");
     transaction.getClientResponse().getHeaders()["X-Content-Transformed"] = "1";
@@ -103,13 +103,13 @@ public:
   }
 
   void
-  consume(const string &data)
+  consume(const string &data) override
   {
     produce(data);
   }
 
   void
-  handleInputComplete()
+  handleInputComplete() override
   {
     Helpers::ContentType content_type = Helpers::getContentType(transaction_);
     if (content_type == Helpers::TEXT_HTML) {
@@ -124,7 +124,7 @@ public:
     setOutputComplete();
   }
 
-  virtual ~SomeTransformationPlugin() {}
+  ~SomeTransformationPlugin() override {}
 
 private:
   Transaction &transaction_;
@@ -140,8 +140,8 @@ public:
     registerHook(HOOK_SEND_RESPONSE_HEADERS);
   }
 
-  virtual void
-  handleSendRequestHeaders(Transaction &transaction)
+  void
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     // Since we can only decompress gzip we will change the accept encoding header
     // to gzip, even if the user cannot accept gziped content we will return to them
@@ -155,8 +155,8 @@ public:
     transaction.resume();
   }
 
-  virtual void
-  handleReadResponseHeaders(Transaction &transaction)
+  void
+  handleReadResponseHeaders(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Determining if we need to add an inflate transformation or a deflate transformation..");
     // We're guaranteed to have been returned either gzipped content or Identity.
@@ -177,8 +177,8 @@ public:
     transaction.resume();
   }
 
-  virtual void
-  handleSendResponseHeaders(Transaction &transaction)
+  void
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     // If the client supported gzip then we can guarantee they are receiving gzip since regardless of the
     // origins content-encoding we returned gzip, so let's make sure the content-encoding header is correctly
diff --git a/example/cppapi/intercept/intercept.cc b/example/cppapi/intercept/intercept.cc
index caff540..b98ba6e 100644
--- a/example/cppapi/intercept/intercept.cc
+++ b/example/cppapi/intercept/intercept.cc
@@ -36,9 +36,9 @@ class Intercept : public InterceptPlugin
 {
 public:
   Intercept(Transaction &transaction) : InterceptPlugin(transaction, InterceptPlugin::SERVER_INTERCEPT) {}
-  void consume(const string &data, InterceptPlugin::RequestDataType type);
-  void handleInputComplete();
-  ~Intercept() { cout << "Shutting down" << endl; }
+  void consume(const string &data, InterceptPlugin::RequestDataType type) override;
+  void handleInputComplete() override;
+  ~Intercept() override { cout << "Shutting down" << endl; }
 };
 
 class InterceptInstaller : public GlobalPlugin
@@ -49,7 +49,7 @@ public:
     GlobalPlugin::registerHook(Plugin::HOOK_READ_REQUEST_HEADERS_PRE_REMAP);
   }
   void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     transaction.addPlugin(new Intercept(transaction));
     cout << "Added intercept" << endl;
diff --git a/example/cppapi/internal_transaction_handling/InternalTransactionHandling.cc b/example/cppapi/internal_transaction_handling/InternalTransactionHandling.cc
index 259b40c..2a7e03e 100644
--- a/example/cppapi/internal_transaction_handling/InternalTransactionHandling.cc
+++ b/example/cppapi/internal_transaction_handling/InternalTransactionHandling.cc
@@ -41,8 +41,8 @@ public:
     registerHook(HOOK_READ_REQUEST_HEADERS_POST_REMAP);
   }
 
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Received a request in handleReadRequestHeadersPostRemap.");
     transaction.resume();
@@ -58,8 +58,8 @@ public:
     registerHook(HOOK_READ_REQUEST_HEADERS_POST_REMAP);
   }
 
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Received a request in handleReadRequestHeadersPostRemap.");
     std::shared_ptr<Mutex> mutex(new Mutex());                                            // required for async operation
@@ -68,7 +68,7 @@ public:
   }
 
   void
-  handleAsyncComplete(AsyncHttpFetch &provider ATSCPPAPI_UNUSED)
+  handleAsyncComplete(AsyncHttpFetch &provider ATSCPPAPI_UNUSED) override
   {
   }
 };
diff --git a/example/cppapi/logger_example/LoggerExample.cc b/example/cppapi/logger_example/LoggerExample.cc
index 14228c1..ba8075f 100644
--- a/example/cppapi/logger_example/LoggerExample.cc
+++ b/example/cppapi/logger_example/LoggerExample.cc
@@ -58,8 +58,8 @@ public:
     registerHook(HOOK_READ_REQUEST_HEADERS_POST_REMAP);
   }
 
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     LOG_DEBUG(log, "handleReadRequestHeadersPostRemap.\n"
                    "\tRequest URL: %s\n"
diff --git a/example/cppapi/multiple_transaction_hooks/MultipleTransactionHookPlugins.cc b/example/cppapi/multiple_transaction_hooks/MultipleTransactionHookPlugins.cc
index 5e97ebc..372a5fe 100644
--- a/example/cppapi/multiple_transaction_hooks/MultipleTransactionHookPlugins.cc
+++ b/example/cppapi/multiple_transaction_hooks/MultipleTransactionHookPlugins.cc
@@ -37,9 +37,9 @@ public:
     std::cout << "Constructed MultipleTransactionHookPluginsOne!" << std::endl;
   }
 
-  virtual ~MultipleTransactionHookPluginsOne() { std::cout << "Destroyed MultipleTransactionHookPluginsOne!" << std::endl; }
+  ~MultipleTransactionHookPluginsOne() override { std::cout << "Destroyed MultipleTransactionHookPluginsOne!" << std::endl; }
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     std::cerr << "MultipleTransactionHookPluginsOne -- Send response headers!" << std::endl;
     transaction.resume();
@@ -56,9 +56,9 @@ public:
     std::cout << "Constructed MultipleTransactionHookPluginsTwo!" << std::endl;
   }
 
-  virtual ~MultipleTransactionHookPluginsTwo() { std::cout << "Destroyed MultipleTransactionHookPluginsTwo!" << std::endl; }
+  ~MultipleTransactionHookPluginsTwo() override { std::cout << "Destroyed MultipleTransactionHookPluginsTwo!" << std::endl; }
   void
-  handleSendRequestHeaders(Transaction &transaction)
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     std::cout << "MultipleTransactionHookPluginsTwo -- Send request headers!" << std::endl;
     some_container_.push_back("We have transaction scoped storage in Transaction Hooks!");
@@ -66,7 +66,7 @@ public:
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     std::cout << "MultipleTransactionHookPluginsTwo -- Send response headers!" << std::endl;
 
@@ -86,8 +86,8 @@ class GlobalHookPlugin : public atscppapi::GlobalPlugin
 {
 public:
   GlobalHookPlugin() { GlobalPlugin::registerHook(HOOK_READ_REQUEST_HEADERS_PRE_REMAP); }
-  virtual void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     std::cout << "Hello from handleReadRequesHeadersPreRemap!" << std::endl;
 
diff --git a/example/cppapi/null_transformation_plugin/NullTransformationPlugin.cc b/example/cppapi/null_transformation_plugin/NullTransformationPlugin.cc
index 6080e93..d85732e 100644
--- a/example/cppapi/null_transformation_plugin/NullTransformationPlugin.cc
+++ b/example/cppapi/null_transformation_plugin/NullTransformationPlugin.cc
@@ -43,32 +43,32 @@ public:
   }
 
   void
-  handleSendRequestHeaders(Transaction &transaction)
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     transaction.getServerRequest().getHeaders()["X-Content-Transformed"] = "1";
     transaction.resume();
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     transaction.getClientResponse().getHeaders()["X-Content-Transformed"] = "1";
     transaction.resume();
   }
 
   void
-  consume(const string &data)
+  consume(const string &data) override
   {
     produce(data);
   }
 
   void
-  handleInputComplete()
+  handleInputComplete() override
   {
     setOutputComplete();
   }
 
-  virtual ~NullTransformationPlugin() {}
+  ~NullTransformationPlugin() override {}
 
 private:
 };
@@ -82,15 +82,15 @@ public:
     registerHook(HOOK_READ_RESPONSE_HEADERS);
   }
 
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     transaction.addPlugin(new NullTransformationPlugin(transaction, TransformationPlugin::REQUEST_TRANSFORMATION));
     transaction.resume();
   }
 
-  virtual void
-  handleReadResponseHeaders(Transaction &transaction)
+  void
+  handleReadResponseHeaders(Transaction &transaction) override
   {
     transaction.addPlugin(new NullTransformationPlugin(transaction, TransformationPlugin::RESPONSE_TRANSFORMATION));
     transaction.resume();
diff --git a/example/cppapi/post_buffer/PostBuffer.cc b/example/cppapi/post_buffer/PostBuffer.cc
index ac20974..941de6b 100644
--- a/example/cppapi/post_buffer/PostBuffer.cc
+++ b/example/cppapi/post_buffer/PostBuffer.cc
@@ -43,19 +43,19 @@ public:
   }
 
   void
-  consume(const string &data)
+  consume(const string &data) override
   {
     buffer_.append(data);
   }
 
   void
-  handleInputComplete()
+  handleInputComplete() override
   {
     produce(buffer_);
     setOutputComplete();
   }
 
-  virtual ~PostBufferTransformationPlugin() {}
+  ~PostBufferTransformationPlugin() override {}
 
 private:
   Transaction &transaction_;
@@ -66,8 +66,8 @@ class GlobalHookPlugin : public GlobalPlugin
 {
 public:
   GlobalHookPlugin() { registerHook(HOOK_READ_REQUEST_HEADERS_POST_REMAP); }
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     cerr << "Read Request Headers Post Remap" << endl;
     cerr << "Path: " << transaction.getClientRequest().getUrl().getPath() << endl;
diff --git a/example/cppapi/remap_plugin/RemapPlugin.cc b/example/cppapi/remap_plugin/RemapPlugin.cc
index be9a56a..455a805 100644
--- a/example/cppapi/remap_plugin/RemapPlugin.cc
+++ b/example/cppapi/remap_plugin/RemapPlugin.cc
@@ -38,7 +38,7 @@ class MyRemapPlugin : public RemapPlugin
 public:
   MyRemapPlugin(void **instance_handle) : RemapPlugin(instance_handle) {}
   Result
-  doRemap(const Url &map_from_url, const Url &map_to_url, Transaction &transaction, bool &redirect)
+  doRemap(const Url &map_from_url, const Url &map_to_url, Transaction &transaction, bool &redirect) override
   {
     Url &request_url = transaction.getClientRequest().getUrl();
     TS_DEBUG(LOG_TAG, "from URL is [%s], to URL is [%s], request URL is [%s]", map_from_url.getUrlString().c_str(),
diff --git a/example/cppapi/serverresponse/ServerResponse.cc b/example/cppapi/serverresponse/ServerResponse.cc
index b75e0c6..5e2e7f8 100644
--- a/example/cppapi/serverresponse/ServerResponse.cc
+++ b/example/cppapi/serverresponse/ServerResponse.cc
@@ -44,7 +44,7 @@ public:
   }
 
   void
-  handleSendRequestHeaders(Transaction &transaction)
+  handleSendRequestHeaders(Transaction &transaction) override
   {
     // Here we can decide to abort the request to the origin (we can do this earlier too)
     // and just send the user an error page.
@@ -62,7 +62,7 @@ public:
   }
 
   void
-  handleReadResponseHeaders(Transaction &transaction)
+  handleReadResponseHeaders(Transaction &transaction) override
   {
     cout << "Hello from handleReadResponseHeaders!" << endl;
     cout << "Server response headers are" << endl;
@@ -73,7 +73,7 @@ public:
   }
 
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     cout << "Hello from handleSendResponseHeaders!" << endl;
     cout << "Client response headers are" << endl;
diff --git a/example/cppapi/stat_example/StatExample.cc b/example/cppapi/stat_example/StatExample.cc
index 97414dd..cbfb5c4 100644
--- a/example/cppapi/stat_example/StatExample.cc
+++ b/example/cppapi/stat_example/StatExample.cc
@@ -55,8 +55,8 @@ public:
     registerHook(HOOK_READ_REQUEST_HEADERS_POST_REMAP);
   }
 
-  virtual void
-  handleReadRequestHeadersPostRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPostRemap(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Received a request, incrementing the counter.");
     stat.increment();
diff --git a/example/cppapi/timeout_example/TimeoutExamplePlugin.cc b/example/cppapi/timeout_example/TimeoutExamplePlugin.cc
index b05dc03..d2b3120 100644
--- a/example/cppapi/timeout_example/TimeoutExamplePlugin.cc
+++ b/example/cppapi/timeout_example/TimeoutExamplePlugin.cc
@@ -39,15 +39,15 @@ public:
     registerHook(HOOK_SEND_RESPONSE_HEADERS);
   }
 
-  virtual void
-  handleSendResponseHeaders(Transaction &transaction)
+  void
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Sending response headers to the client, status=%d", transaction.getClientResponse().getStatusCode());
     transaction.resume();
   }
 
-  virtual void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     TS_DEBUG(TAG, "Setting all timeouts to 1ms, this will likely cause the transaction to receive a 504.");
     transaction.setTimeout(Transaction::TIMEOUT_CONNECT, 1);
diff --git a/example/cppapi/transactionhook/TransactionHookPlugin.cc b/example/cppapi/transactionhook/TransactionHookPlugin.cc
index 64e41bb..c89dec0 100644
--- a/example/cppapi/transactionhook/TransactionHookPlugin.cc
+++ b/example/cppapi/transactionhook/TransactionHookPlugin.cc
@@ -37,13 +37,13 @@ public:
     TransactionPlugin::registerHook(HOOK_SEND_RESPONSE_HEADERS);
     std::cout << "Constructed!" << std::endl;
   }
-  virtual ~TransactionHookPlugin()
+  ~TransactionHookPlugin() override
   {
     delete[] char_ptr_; // cleanup
     std::cout << "Destroyed!" << std::endl;
   }
   void
-  handleSendResponseHeaders(Transaction &transaction)
+  handleSendResponseHeaders(Transaction &transaction) override
   {
     std::cout << "Send response headers!" << std::endl;
     transaction.resume();
@@ -57,8 +57,8 @@ class GlobalHookPlugin : public atscppapi::GlobalPlugin
 {
 public:
   GlobalHookPlugin() { GlobalPlugin::registerHook(HOOK_READ_REQUEST_HEADERS_PRE_REMAP); }
-  virtual void
-  handleReadRequestHeadersPreRemap(Transaction &transaction)
+  void
+  handleReadRequestHeadersPreRemap(Transaction &transaction) override
   {
     std::cout << "Hello from handleReadRequesHeadersPreRemap!" << std::endl;
     transaction.addPlugin(new TransactionHookPlugin(transaction));
diff --git a/iocore/cache/CachePages.cc b/iocore/cache/CachePages.cc
index 42f6721..ee26ebb 100644
--- a/iocore/cache/CachePages.cc
+++ b/iocore/cache/CachePages.cc
@@ -152,7 +152,7 @@ struct ShowCache : public ShowCont {
     SET_HANDLER(&ShowCache::showMain);
   }
 
-  ~ShowCache()
+  ~ShowCache() override
   {
     if (show_cache_urlstrs)
       delete[] show_cache_urlstrs;
diff --git a/iocore/cache/CachePagesInternal.cc b/iocore/cache/CachePagesInternal.cc
index 34a1771..7cf8de1 100644
--- a/iocore/cache/CachePagesInternal.cc
+++ b/iocore/cache/CachePagesInternal.cc
@@ -49,7 +49,7 @@ struct ShowCacheInternal : public ShowCont {
     SET_HANDLER(&ShowCacheInternal::showMain);
   }
 
-  ~ShowCacheInternal() {}
+  ~ShowCacheInternal() override {}
 };
 extern ShowCacheInternal *theshowcacheInternal;
 Action *register_ShowCacheInternal(Continuation *c, HTTPHdr *h);
diff --git a/iocore/cache/RamCacheCLFUS.cc b/iocore/cache/RamCacheCLFUS.cc
index be7ae6b..6d00449 100644
--- a/iocore/cache/RamCacheCLFUS.cc
+++ b/iocore/cache/RamCacheCLFUS.cc
@@ -76,12 +76,12 @@ struct RamCacheCLFUS : public RamCache {
   int64_t objects;
 
   // returns 1 on found/stored, 0 on not found/stored, if provided auxkey1 and auxkey2 must match
-  int get(INK_MD5 *key, Ptr<IOBufferData> *ret_data, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0);
-  int put(INK_MD5 *key, IOBufferData *data, uint32_t len, bool copy = false, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0);
-  int fixup(const INK_MD5 *key, uint32_t old_auxkey1, uint32_t old_auxkey2, uint32_t new_auxkey1, uint32_t new_auxkey2);
-  int64_t size() const;
+  int get(INK_MD5 *key, Ptr<IOBufferData> *ret_data, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0) override;
+  int put(INK_MD5 *key, IOBufferData *data, uint32_t len, bool copy = false, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0) override;
+  int fixup(const INK_MD5 *key, uint32_t old_auxkey1, uint32_t old_auxkey2, uint32_t new_auxkey1, uint32_t new_auxkey2) override;
+  int64_t size() const override;
 
-  void init(int64_t max_bytes, Vol *vol);
+  void init(int64_t max_bytes, Vol *vol) override;
 
   // private
   Vol *vol; // for stats
diff --git a/iocore/cache/RamCacheLRU.cc b/iocore/cache/RamCacheLRU.cc
index 74c83c1..31c3d8e 100644
--- a/iocore/cache/RamCacheLRU.cc
+++ b/iocore/cache/RamCacheLRU.cc
@@ -40,12 +40,12 @@ struct RamCacheLRU : public RamCache {
   int64_t objects;
 
   // returns 1 on found/stored, 0 on not found/stored, if provided auxkey1 and auxkey2 must match
-  int get(INK_MD5 *key, Ptr<IOBufferData> *ret_data, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0);
-  int put(INK_MD5 *key, IOBufferData *data, uint32_t len, bool copy = false, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0);
-  int fixup(const INK_MD5 *key, uint32_t old_auxkey1, uint32_t old_auxkey2, uint32_t new_auxkey1, uint32_t new_auxkey2);
-  int64_t size() const;
+  int get(INK_MD5 *key, Ptr<IOBufferData> *ret_data, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0) override;
+  int put(INK_MD5 *key, IOBufferData *data, uint32_t len, bool copy = false, uint32_t auxkey1 = 0, uint32_t auxkey2 = 0) override;
+  int fixup(const INK_MD5 *key, uint32_t old_auxkey1, uint32_t old_auxkey2, uint32_t new_auxkey1, uint32_t new_auxkey2) override;
+  int64_t size() const override;
 
-  void init(int64_t max_bytes, Vol *vol);
+  void init(int64_t max_bytes, Vol *vol) override;
 
   // private
   uint16_t *seen;
diff --git a/iocore/cluster/ClusterAPI.cc b/iocore/cluster/ClusterAPI.cc
index 8926deb..1eee14d 100644
--- a/iocore/cluster/ClusterAPI.cc
+++ b/iocore/cluster/ClusterAPI.cc
@@ -94,7 +94,7 @@ public:
   {
     SET_HANDLER((MachineStatusSMHandler)&MachineStatusSM::MachineStatusSMEvent);
   }
-  ~MachineStatusSM() {}
+  ~MachineStatusSM() override {}
   int MachineStatusSMEvent(Event *e, void *d);
 
 private:
@@ -197,7 +197,7 @@ public:
   {
     SET_HANDLER((ClusterAPIPeriodicSMHandler)&ClusterAPIPeriodicSM::ClusterAPIPeriodicSMEvent);
   }
-  ~ClusterAPIPeriodicSM() {}
+  ~ClusterAPIPeriodicSM() override {}
   int ClusterAPIPeriodicSMEvent(int, void *);
   MachineStatusSM *GetNextSM();
 
diff --git a/iocore/hostdb/HostDB.cc b/iocore/hostdb/HostDB.cc
index f31fa37..4ff5a01 100644
--- a/iocore/hostdb/HostDB.cc
+++ b/iocore/hostdb/HostDB.cc
@@ -271,7 +271,7 @@ struct HostDBSync : public HostDBBackgroundTask {
   HostDBSync(int frequency, std::string storage_path, std::string full_path)
     : HostDBBackgroundTask(frequency), storage_path(storage_path), full_path(full_path){};
   int
-  sync_event(int, void *)
+  sync_event(int, void *) override
   {
     SET_HANDLER(&HostDBSync::wait_event);
     start_time = Thread::get_hrtime();
diff --git a/iocore/hostdb/test_RefCountCache.cc b/iocore/hostdb/test_RefCountCache.cc
index f1d85c8..a709690 100644
--- a/iocore/hostdb/test_RefCountCache.cc
+++ b/iocore/hostdb/test_RefCountCache.cc
@@ -59,7 +59,7 @@ public:
 
   // Really free the memory, we can use asan leak detection to verify it was freed
   void
-  free()
+  free() override
   {
     this->idx = -1;
     items_freed.insert(this);
diff --git a/iocore/net/UnixNetProcessor.cc b/iocore/net/UnixNetProcessor.cc
index 2485a54..905d268 100644
--- a/iocore/net/UnixNetProcessor.cc
+++ b/iocore/net/UnixNetProcessor.cc
@@ -379,7 +379,7 @@ struct CheckConnect : public Continuation {
     reader = buf->alloc_reader();
   }
 
-  ~CheckConnect()
+  ~CheckConnect() override
   {
     buf->dealloc_all_readers();
     buf->clear();
diff --git a/iocore/net/UnixUDPNet.cc b/iocore/net/UnixUDPNet.cc
index b05023a..da80c8d 100644
--- a/iocore/net/UnixUDPNet.cc
+++ b/iocore/net/UnixUDPNet.cc
@@ -176,7 +176,7 @@ class UDPReadContinuation : public Continuation
 public:
   UDPReadContinuation(Event *completionToken);
   UDPReadContinuation();
-  ~UDPReadContinuation();
+  ~UDPReadContinuation() override;
   inline void free(void);
   inline void init_token(Event *completionToken);
   inline void init_read(int fd, IOBufferBlock *buf, int len, struct sockaddr *fromaddr, socklen_t *fromaddrlen);
diff --git a/lib/records/RecProcess.cc b/lib/records/RecProcess.cc
index 7f02b6f..930a01a 100644
--- a/lib/records/RecProcess.cc
+++ b/lib/records/RecProcess.cc
@@ -179,7 +179,7 @@ struct sync_cont : public Continuation {
     m_tb = new TextBuffer(65536);
   }
 
-  ~sync_cont()
+  ~sync_cont() override
   {
     if (m_tb != nullptr) {
       delete m_tb;
diff --git a/lib/ts/IpMap.cc b/lib/ts/IpMap.cc
index da2b73b..81e71a9 100644
--- a/lib/ts/IpMap.cc
+++ b/lib/ts/IpMap.cc
@@ -748,21 +748,21 @@ namespace detail
       ats_ip4_set(ats_ip_sa_cast(&_sa._max), htonl(max));
     }
     /// @return The minimum value of the interval.
-    virtual sockaddr const *
-    min() const
+    sockaddr const *
+    min() const override
     {
       return ats_ip_sa_cast(&_sa._min);
     }
     /// @return The maximum value of the interval.
-    virtual sockaddr const *
-    max() const
+    sockaddr const *
+    max() const override
     {
       return ats_ip_sa_cast(&_sa._max);
     }
     /// Set the client data.
     self &
     setData(void *data ///< Client data.
-            )
+            ) override
     {
       _data = data;
       return *this;
@@ -902,21 +902,21 @@ namespace detail
     {
     }
     /// @return The minimum value of the interval.
-    virtual sockaddr const *
-    min() const
+    sockaddr const *
+    min() const override
     {
       return ats_ip_sa_cast(&_min);
     }
     /// @return The maximum value of the interval.
-    virtual sockaddr const *
-    max() const
+    sockaddr const *
+    max() const override
     {
       return ats_ip_sa_cast(&_max);
     }
     /// Set the client data.
     self &
     setData(void *data ///< Client data.
-            )
+            ) override
     {
       _data = data;
       return *this;
diff --git a/lib/ts/test_Ptr.cc b/lib/ts/test_Ptr.cc
index eb79036..9392bf2 100644
--- a/lib/ts/test_Ptr.cc
+++ b/lib/ts/test_Ptr.cc
@@ -24,7 +24,7 @@
 
 struct PtrObject : RefCountObj {
   PtrObject(unsigned *_c) : count(_c) { ++(*count); }
-  ~PtrObject() { --(*count); }
+  ~PtrObject() override { --(*count); }
   unsigned *count;
 };
 
diff --git a/mgmt/ProxyConfig.cc b/mgmt/ProxyConfig.cc
index 23cf750..048bf98 100644
--- a/mgmt/ProxyConfig.cc
+++ b/mgmt/ProxyConfig.cc
@@ -254,7 +254,7 @@ struct RegressionConfig : public ConfigInfo {
     ink_atomic_increment(&nobjects, 1);
   }
 
-  ~RegressionConfig()
+  ~RegressionConfig() override
   {
     TestBox box(this->test, this->pstatus);
 
diff --git a/plugins/experimental/balancer/hash.cc b/plugins/experimental/balancer/hash.cc
index bc33bb3..2e7da28 100644
--- a/plugins/experimental/balancer/hash.cc
+++ b/plugins/experimental/balancer/hash.cc
@@ -154,7 +154,7 @@ struct HashBalancer : public BalancerInstance {
 
   HashBalancer() { this->hash_parts.push_back(HashTxnUrl); }
   void
-  push_target(const BalancerTarget &target)
+  push_target(const BalancerTarget &target) override
   {
     for (unsigned i = 0; i < iterations; ++i) {
       this->hash_ring.insert(std::make_pair(md5_key(target, i), target));
@@ -162,7 +162,7 @@ struct HashBalancer : public BalancerInstance {
   }
 
   const BalancerTarget &
-  balance(TSHttpTxn txn, TSRemapRequestInfo *rri)
+  balance(TSHttpTxn txn, TSRemapRequestInfo *rri) override
   {
     md5_key key;
     MD5_CTX ctx;
diff --git a/plugins/experimental/balancer/roundrobin.cc b/plugins/experimental/balancer/roundrobin.cc
index 6529644..2e1aa83 100644
--- a/plugins/experimental/balancer/roundrobin.cc
+++ b/plugins/experimental/balancer/roundrobin.cc
@@ -33,13 +33,13 @@ namespace
 struct RoundRobinBalancer : public BalancerInstance {
   RoundRobinBalancer() : targets(), next(0) {}
   void
-  push_target(const BalancerTarget &target)
+  push_target(const BalancerTarget &target) override
   {
     this->targets.push_back(target);
   }
 
   const BalancerTarget &
-  balance(TSHttpTxn, TSRemapRequestInfo *)
+  balance(TSHttpTxn, TSRemapRequestInfo *) override
   {
     return this->targets[++next % this->targets.size()];
   }
diff --git a/plugins/experimental/cache_promote/cache_promote.cc b/plugins/experimental/cache_promote/cache_promote.cc
index 8245605..f60a9e3 100644
--- a/plugins/experimental/cache_promote/cache_promote.cc
+++ b/plugins/experimental/cache_promote/cache_promote.cc
@@ -116,20 +116,20 @@ private:
 class ChancePolicy : public PromotionPolicy
 {
 public:
-  bool doPromote(TSHttpTxn /* txnp ATS_UNUSED */)
+  bool doPromote(TSHttpTxn /* txnp ATS_UNUSED */) override
   {
     TSDebug(PLUGIN_NAME, "ChancePolicy::doPromote(%f)", getSample());
     return true;
   }
 
   void
-  usage() const
+  usage() const override
   {
     TSError("[%s] Usage: @plugin=%s.so @pparam=--policy=chance @pparam=--sample=<x>%%", PLUGIN_NAME, PLUGIN_NAME);
   }
 
   const char *
-  policyName() const
+  policyName() const override
   {
     return "chance";
   }
@@ -194,7 +194,7 @@ class LRUPolicy : public PromotionPolicy
 {
 public:
   LRUPolicy() : PromotionPolicy(), _buckets(1000), _hits(10), _lock(TSMutexCreate()), _list_size(0), _freelist_size(0) {}
-  ~LRUPolicy()
+  ~LRUPolicy() override
   {
     TSDebug(PLUGIN_NAME, "deleting LRUPolicy object");
     TSMutexLock(_lock);
@@ -210,7 +210,7 @@ public:
   }
 
   bool
-  parseOption(int opt, char *optarg)
+  parseOption(int opt, char *optarg) override
   {
     switch (opt) {
     case 'b':
@@ -237,7 +237,7 @@ public:
   }
 
   bool
-  doPromote(TSHttpTxn txnp)
+  doPromote(TSHttpTxn txnp) override
   {
     LRUHash hash;
     LRUMap::iterator map_it;
@@ -318,14 +318,14 @@ public:
   }
 
   void
-  usage() const
+  usage() const override
   {
     TSError("[%s] Usage: @plugin=%s.so @pparam=--policy=lru @pparam=--buckets=<n> --hits=<m> --sample=<x>", PLUGIN_NAME,
             PLUGIN_NAME);
   }
 
   const char *
-  policyName() const
+  policyName() const override
   {
     return "LRU";
   }
diff --git a/plugins/experimental/stream_editor/stream_editor.cc b/plugins/experimental/stream_editor/stream_editor.cc
index a36cb77..01286c8 100644
--- a/plugins/experimental/stream_editor/stream_editor.cc
+++ b/plugins/experimental/stream_editor/stream_editor.cc
@@ -207,9 +207,9 @@ class rxscope : public scope_t
 {
 private:
   regex_t rx;
-  virtual bool
+  bool
 
-  match(const char *str) const
+  match(const char *str) const override
   {
     return (regexec(&rx, str, 0, nullptr, 0) == 0) ? true : false;
   }
@@ -228,7 +228,7 @@ public:
     TSfree(str);
   }
 
-  virtual ~rxscope() { regfree(&rx); }
+  ~rxscope() override { regfree(&rx); }
 };
 
 class strscope : public scope_t
@@ -236,16 +236,16 @@ class strscope : public scope_t
 private:
   const bool icase;
   char *str;
-  virtual bool
+  bool
 
-  match(const char *p) const
+  match(const char *p) const override
   {
     return ((icase ? strncasecmp : strncmp)(str, p, strlen(str)) == 0) ? true : false;
   }
 
 public:
   strscope(const bool u, const bool i, const char *pattern, int len) : scope_t(u), icase(i) { str = TSstrndup(pattern, len); }
-  virtual ~strscope()
+  ~strscope() override
   {
     if (str) {
       TSfree(str);
@@ -268,8 +268,8 @@ class strmatch : public match_t
   const size_t slen;
 
 public:
-  virtual bool
-  find(const char *buf, size_t len, size_t &found, size_t &found_len, const char *to, std::string &repl) const
+  bool
+  find(const char *buf, size_t len, size_t &found, size_t &found_len, const char *to, std::string &repl) const override
   {
     const char *match = icase ? strcasestr(buf, str) : strstr(buf, str);
     if (match) {
@@ -283,15 +283,15 @@ public:
   }
 
   strmatch(const bool i, const char *pattern, int len) : icase(i), slen(len) { str = TSstrndup(pattern, len); }
-  virtual ~strmatch()
+  ~strmatch() override
   {
     if (str) {
       TSfree(str);
     }
   }
 
-  virtual size_t
-  cont_size() const
+  size_t
+  cont_size() const override
   {
     return slen;
   }
@@ -303,8 +303,8 @@ class rxmatch : public match_t
   regex_t rx;
 
 public:
-  virtual bool
-  find(const char *buf, size_t len, size_t &found, size_t &found_len, const char *tmpl, std::string &repl) const
+  bool
+  find(const char *buf, size_t len, size_t &found, size_t &found_len, const char *tmpl, std::string &repl) const override
   {
     regmatch_t pmatch[MAX_RX_MATCH];
     if (regexec(&rx, buf, MAX_RX_MATCH, pmatch, REG_NOTEOL) == 0) {
@@ -343,8 +343,8 @@ public:
     }
   }
 
-  virtual size_t
-  cont_size() const
+  size_t
+  cont_size() const override
   {
     return match_len;
   }
@@ -362,7 +362,7 @@ public:
     TSfree(str);
   }
 
-  virtual ~rxmatch() { regfree(&rx); }
+  ~rxmatch() override { regfree(&rx); }
 };
 
 #define PARSE_VERIFY(line, x, str) \
diff --git a/plugins/experimental/webp_transform/ImageTransform.cc b/plugins/experimental/webp_transform/ImageTransform.cc
index d881a71..fe37f8c 100644
--- a/plugins/experimental/webp_transform/ImageTransform.cc
+++ b/plugins/experimental/webp_transform/ImageTransform.cc
@@ -43,7 +43,7 @@ public:
   }
 
   void
-  handleReadResponseHeaders(Transaction &transaction)
+  handleReadResponseHeaders(Transaction &transaction) override
   {
     transaction.getServerResponse().getHeaders()["Content-Type"] = "image/webp";
     transaction.getServerResponse().getHeaders()["Vary"]         = "Content-Type"; // to have a separate cache entry.
@@ -53,13 +53,13 @@ public:
   }
 
   void
-  consume(const string &data)
+  consume(const string &data) override
   {
     _img.write(data.data(), data.size());
   }
 
   void
-  handleInputComplete()
+  handleInputComplete() override
   {
     string input_data = _img.str();
     Blob input_blob(input_data.data(), input_data.length());
@@ -75,7 +75,7 @@ public:
     setOutputComplete();
   }
 
-  virtual ~ImageTransform() {}
+  ~ImageTransform() override {}
 
 private:
   std::stringstream _img;
@@ -85,8 +85,8 @@ class GlobalHookPlugin : public GlobalPlugin
 {
 public:
   GlobalHookPlugin() { registerHook(HOOK_READ_RESPONSE_HEADERS); }
-  virtual void
-  handleReadResponseHeaders(Transaction &transaction)
+  void
+  handleReadResponseHeaders(Transaction &transaction) override
   {
     string ctype      = transaction.getServerResponse().getHeaders().values("Content-Type");
     string user_agent = transaction.getServerRequest().getHeaders().values("User-Agent");
diff --git a/proxy/ControlBase.cc b/proxy/ControlBase.cc
index 7f93f4f..16df04a 100644
--- a/proxy/ControlBase.cc
+++ b/proxy/ControlBase.cc
@@ -74,10 +74,10 @@ struct TimeMod : public ControlBase::Modifier {
 
   static const char *const NAME;
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
-  virtual void print(FILE *f) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
+  void print(FILE *f) const override;
   static TimeMod *make(char *value, const char **error);
   static const char *timeOfDayToSeconds(const char *time_str, time_t *seconds);
 };
@@ -180,9 +180,9 @@ struct PortMod : public ControlBase::Modifier {
 
   static const char *const NAME;
 
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
-  virtual void print(FILE *f) const;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
+  void print(FILE *f) const override;
 
   static PortMod *make(char *value, const char **error);
 };
@@ -244,9 +244,9 @@ struct IPortMod : public ControlBase::Modifier {
 
   IPortMod(int port);
 
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
-  virtual void print(FILE *f) const;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
+  void print(FILE *f) const override;
   static IPortMod *make(char *value, const char **error);
 };
 
@@ -292,10 +292,10 @@ struct SrcIPMod : public ControlBase::Modifier {
 
   static const char *const NAME;
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
-  virtual void print(FILE *f) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
+  void print(FILE *f) const override;
   static SrcIPMod *make(char *value, const char **error);
 };
 
@@ -343,10 +343,10 @@ struct SchemeMod : public ControlBase::Modifier {
 
   SchemeMod(int scheme);
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
-  virtual void print(FILE *f) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
+  void print(FILE *f) const override;
 
   const char *getWksText() const;
 
@@ -405,10 +405,10 @@ struct TextMod : public ControlBase::Modifier {
   ts::Buffer text;
 
   TextMod();
-  ~TextMod();
+  ~TextMod() override;
 
   // Calls name() which the subclass must provide.
-  virtual void print(FILE *f) const;
+  void print(FILE *f) const override;
 
   // Copy the given NUL-terminated string to the text buffer.
   void set(const char *value);
@@ -438,13 +438,13 @@ TextMod::set(const char *value)
 struct MultiTextMod : public ControlBase::Modifier {
   Vec<ts::Buffer> text_vec;
   MultiTextMod();
-  ~MultiTextMod();
+  ~MultiTextMod() override;
 
   // Copy the value to the MultiTextMod buffer.
   void set(char *value);
 
   // Calls name() which the subclass must provide.
-  virtual void print(FILE *f) const;
+  void print(FILE *f) const override;
 };
 
 MultiTextMod::MultiTextMod()
@@ -479,9 +479,9 @@ MultiTextMod::set(char *value)
 struct MethodMod : public TextMod {
   static const char *const NAME;
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
 
   static MethodMod *make(char *value, const char **error);
 };
@@ -515,9 +515,9 @@ MethodMod::make(char *value, const char **)
 struct PrefixMod : public TextMod {
   static const char *const NAME;
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
   static PrefixMod *make(char *value, const char **error);
 };
 
@@ -563,9 +563,9 @@ PrefixMod::make(char *value, const char ** /* error ATS_UNUSED */)
 struct SuffixMod : public MultiTextMod {
   static const char *const NAME;
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
   static SuffixMod *make(char *value, const char **error);
 };
 const char *const SuffixMod::NAME = "Suffix";
@@ -611,9 +611,9 @@ SuffixMod::make(char *value, const char ** /* error ATS_UNUSED */)
 struct TagMod : public TextMod {
   static const char *const NAME;
 
-  virtual Type type() const;
-  virtual const char *name() const;
-  virtual bool check(HttpRequestData *req) const;
+  Type type() const override;
+  const char *name() const override;
+  bool check(HttpRequestData *req) const override;
   static TagMod *make(char *value, const char **error);
 };
 const char *const TagMod::NAME = "Tag";
@@ -645,23 +645,23 @@ struct InternalMod : public ControlBase::Modifier {
   bool flag;
   static const char *const NAME;
 
-  virtual Type
-  type() const
+  Type
+  type() const override
   {
     return MOD_INTERNAL;
   }
-  virtual const char *
-  name() const
+  const char *
+  name() const override
   {
     return NAME;
   }
-  virtual bool
-  check(HttpRequestData *req) const
+  bool
+  check(HttpRequestData *req) const override
   {
     return req->internal_txn == flag;
   }
-  virtual void
-  print(FILE *f) const
+  void
+  print(FILE *f) const override
   {
     fprintf(f, "%s=%s  ", this->name(), flag ? "true" : "false");
   }
diff --git a/proxy/Main.cc b/proxy/Main.cc
index b251557..1b561fb 100644
--- a/proxy/Main.cc
+++ b/proxy/Main.cc
@@ -292,7 +292,7 @@ public:
     baseline_taken = 0;
   }
 
-  ~TrackerContinuation() { mutex = nullptr; }
+  ~TrackerContinuation() override { mutex = nullptr; }
   int
   periodic(int event, Event * /* e ATS_UNUSED */)
   {
@@ -357,7 +357,7 @@ public:
     memset(&_usage, 0, sizeof(_usage));
     SET_HANDLER(&MemoryLimit::periodic);
   }
-  ~MemoryLimit() { mutex = nullptr; }
+  ~MemoryLimit() override { mutex = nullptr; }
   int
   periodic(int event, Event *e)
   {
diff --git a/proxy/PluginVC.cc b/proxy/PluginVC.cc
index 9e7249a..7a377e7 100644
--- a/proxy/PluginVC.cc
+++ b/proxy/PluginVC.cc
@@ -1243,7 +1243,7 @@ class PVCTestDriver : public NetTestDriver
 {
 public:
   PVCTestDriver();
-  ~PVCTestDriver();
+  ~PVCTestDriver() override;
 
   void start_tests(RegressionTest *r_arg, int *pstatus_arg);
   void run_next_test();
diff --git a/proxy/RegressionSM.cc b/proxy/RegressionSM.cc
index 84d75d4..b31368e 100644
--- a/proxy/RegressionSM.cc
+++ b/proxy/RegressionSM.cc
@@ -252,8 +252,8 @@ RegressionSM::RegressionSM(const RegressionSM &ao) : Continuation(ao)
 }
 
 struct ReRegressionSM : public RegressionSM {
-  virtual void
-  run()
+  void
+  run() override
   {
     if (time(nullptr) < 1) { // example test
       rprintf(t, "impossible");
@@ -263,8 +263,8 @@ struct ReRegressionSM : public RegressionSM {
     }
   }
   ReRegressionSM(RegressionTest *at) : RegressionSM(at) {}
-  virtual RegressionSM *
-  clone()
+  RegressionSM *
+  clone() override
   {
     return new ReRegressionSM(*this);
   }
diff --git a/proxy/SocksProxy.cc b/proxy/SocksProxy.cc
index 2fbaf71..d995367 100644
--- a/proxy/SocksProxy.cc
+++ b/proxy/SocksProxy.cc
@@ -69,7 +69,7 @@ struct SocksProxy : public Continuation {
       pending_action(nullptr)
   {
   }
-  ~SocksProxy() {}
+  ~SocksProxy() override {}
   // int startEvent(int event, void * data);
   int mainEvent(int event, void *data);
   int setupHttpRequest(unsigned char *p);
diff --git a/proxy/shared/UglyLogStubs.cc b/proxy/shared/UglyLogStubs.cc
index 916636f..c083af4 100644
--- a/proxy/shared/UglyLogStubs.cc
+++ b/proxy/shared/UglyLogStubs.cc
@@ -38,8 +38,8 @@ int fds_limit = 8000;
 
 class FakeUDPNetProcessor : public UDPNetProcessor
 {
-  virtual int
-  start(int, size_t)
+  int
+  start(int, size_t) override
   {
     ink_release_assert(false);
     return 0;

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

Mime
View raw message