You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2017/05/11 03:00:27 UTC

[trafficserver] branch master updated: coverity 1196433 : deadcode is being reported this fix it to try to clarify that the code is not dead

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

sorber 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  7ec886a   coverity 1196433 : deadcode is being reported this fix it to try to clarify that the code is not dead
7ec886a is described below

commit 7ec886ae98de75eac43a1fa0476532dac267f5f7
Author: Jason Kenny <dr...@live.com>
AuthorDate: Wed May 10 15:33:11 2017 -0400

    coverity 1196433 : deadcode is being reported
    this fix it to try to clarify that the code is not dead
---
 iocore/cache/CacheHosting.cc | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/iocore/cache/CacheHosting.cc b/iocore/cache/CacheHosting.cc
index 6952205..ac67e0a 100644
--- a/iocore/cache/CacheHosting.cc
+++ b/iocore/cache/CacheHosting.cc
@@ -245,14 +245,14 @@ CacheHostTable::BuildTableFromString(const char *config_file_path, char *file_bu
   // Table build locals
   Tokenizer bufTok("\n");
   tok_iter_state i_state;
-  const char *tmp;
-  matcher_line *first = nullptr;
-  matcher_line *current;
-  matcher_line *last = nullptr;
-  int line_num       = 0;
-  int second_pass    = 0;
-  int numEntries     = 0;
-  const char *errPtr = nullptr;
+  const char *tmp       = nullptr;
+  matcher_line *first   = nullptr;
+  matcher_line *current = nullptr;
+  matcher_line *last    = nullptr;
+  int line_num          = 0;
+  int second_pass       = 0;
+  int numEntries        = 0;
+  const char *errPtr    = nullptr;
 
   // type counts
   int hostDomain = 0;
@@ -301,7 +301,8 @@ CacheHostTable::BuildTableFromString(const char *config_file_path, char *file_bu
 
         if (first == nullptr) {
           ink_assert(last == nullptr);
-          first = last = current;
+          first = current;
+          last  = current;
         } else {
           last->next = current;
           last       = current;

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