[trafficserver] branch master updated: i4637: A tiny bit of cleanup.

[amc]

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

amc 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 fe70d57  i4637: A tiny bit of cleanup.
fe70d57 is described below

commit fe70d57b3dffa6d00b8e1ef8984e031cc4e7b3ec
Author: Alan M. Carroll <a...@apache.org>
AuthorDate: Tue Jan 15 13:17:44 2019 -0600

    i4637: A tiny bit of cleanup.
---
 iocore/net/SSLCertLookup.cc | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/iocore/net/SSLCertLookup.cc b/iocore/net/SSLCertLookup.cc
index 508ea2f..f94f515 100644
--- a/iocore/net/SSLCertLookup.cc
+++ b/iocore/net/SSLCertLookup.cc
@@ -54,12 +54,12 @@ struct SSLAddressLookupKey {
     // different matches if there is a certificate on the port.
 
     ts::FixedBufferWriter w{key, sizeof(key)};
-    w.print("{}", ts::bwf::Hex_Dump(ip)); // dump raw bytes in hex, don't 
format as IP address.
+    w.print("{}", ts::bwf::Hex_Dump(ip)); // dump as raw hex bytes, don't 
format as IP address.
     if (in_port_t port = ip.host_order_port(); port) {
-      sep = unsigned(w.size());
+      sep = static_cast<unsigned char>(w.size());
       w.print(".{:x}", port);
     }
-    w.print("\0");
+    w.write('\0'); // force C-string termination.
   }
 
   const char *