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>'].