You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by wk...@apache.org on 2020/11/19 17:33:10 UTC

[trafficserver] branch master updated: Remove unnecessary cast from ReverseProxy. (#7329)

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

wkaras 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 b21ecc8  Remove unnecessary cast from ReverseProxy. (#7329)
b21ecc8 is described below

commit b21ecc8f949344080c83d4c52caf37246968126e
Author: Walt Karas <wk...@verizonmedia.com>
AuthorDate: Thu Nov 19 11:32:54 2020 -0600

    Remove unnecessary cast from ReverseProxy. (#7329)
---
 proxy/ReverseProxy.cc | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/proxy/ReverseProxy.cc b/proxy/ReverseProxy.cc
index 22b44af..c55df4a 100644
--- a/proxy/ReverseProxy.cc
+++ b/proxy/ReverseProxy.cc
@@ -104,20 +104,15 @@ response_url_remap(HTTPHdr *response_header, UrlRewrite *table)
 //
 
 /** Used to read the remap.config file after the manager signals a change. */
-struct UR_UpdateContinuation;
-using UR_UpdContHandler = int (UR_UpdateContinuation::*)(int, void *);
 struct UR_UpdateContinuation : public Continuation {
   int
   file_update_handler(int /* etype ATS_UNUSED */, void * /* data ATS_UNUSED */)
   {
-    (void)reloadUrlRewrite();
+    static_cast<void>(reloadUrlRewrite());
     delete this;
     return EVENT_DONE;
   }
-  UR_UpdateContinuation(Ptr<ProxyMutex> &m) : Continuation(m)
-  {
-    SET_HANDLER((UR_UpdContHandler)&UR_UpdateContinuation::file_update_handler);
-  }
+  UR_UpdateContinuation(Ptr<ProxyMutex> &m) : Continuation(m) { SET_HANDLER(&UR_UpdateContinuation::file_update_handler); }
 };
 
 bool