You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2017/10/12 15:33:55 UTC

[trafficserver] branch 7.1.x updated: remove unnecessary assertion for "alternate"

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

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/7.1.x by this push:
     new 6dfd927  remove unnecessary assertion for "alternate"
6dfd927 is described below

commit 6dfd927887e2db9c8862e55455c573f73190637a
Author: Fei Deng <fe...@oath.com>
AuthorDate: Tue Sep 5 16:08:57 2017 -0500

    remove unnecessary assertion for "alternate"
    
    (cherry picked from commit a5fd988170fdacb7df3fad1b2d7cd908a98c28dd)
    
     Conflicts:
    	iocore/cache/CacheRead.cc
    
    Trivially resolved, because of clang-tidy being run on master.
---
 iocore/cache/CacheRead.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/iocore/cache/CacheRead.cc b/iocore/cache/CacheRead.cc
index a9986bd..6b2fbee 100644
--- a/iocore/cache/CacheRead.cc
+++ b/iocore/cache/CacheRead.cc
@@ -245,7 +245,7 @@ CacheVC::openReadChooseWriter(int /* event ATS_UNUSED */, Event * /* e ATS_UNUSE
           continue;
         }
       }
-      ink_assert(w->alternate.valid());
+
       if (w->alternate.valid())
         vector.insert(&w->alternate, alt_ndx);
     }

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