You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by dm...@apache.org on 2024/05/09 07:31:28 UTC
(trafficserver) branch master updated: libswoc: Remove unused variables from some unit tests. (#11326)
This is an automated email from the ASF dual-hosted git repository.
dmeden 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 f2b9fc4cb2 libswoc: Remove unused variables from some unit tests. (#11326)
f2b9fc4cb2 is described below
commit f2b9fc4cb2b1e31beee41de70a6e55c09403151b
Author: Damian Meden <dm...@apache.org>
AuthorDate: Thu May 9 09:31:21 2024 +0200
libswoc: Remove unused variables from some unit tests. (#11326)
---
lib/swoc/unit_tests/test_TextView.cc | 1 -
lib/swoc/unit_tests/test_ip.cc | 1 -
2 files changed, 2 deletions(-)
diff --git a/lib/swoc/unit_tests/test_TextView.cc b/lib/swoc/unit_tests/test_TextView.cc
index 20c1797deb..14995948b3 100644
--- a/lib/swoc/unit_tests/test_TextView.cc
+++ b/lib/swoc/unit_tests/test_TextView.cc
@@ -400,7 +400,6 @@ TEST_CASE("TextView Affixes", "[libswoc][TextView]") {
// Checking that constexpr works for this constructor as long as npos isn't used.
static constexpr TextView ctv2{"http://delain.nl/albums/Interlude.html", 38};
- TextView ctv4{"http://delain.nl/albums/Interlude.html", 38};
// This doesn't compile because it causes strlen to be called which isn't constexpr compatible.
// static constexpr TextView ctv3 {"http://delain.nl/albums/Interlude.html", TextView::npos};
// This works because it's not constexpr.
diff --git a/lib/swoc/unit_tests/test_ip.cc b/lib/swoc/unit_tests/test_ip.cc
index d240f97d26..c25b88150e 100644
--- a/lib/swoc/unit_tests/test_ip.cc
+++ b/lib/swoc/unit_tests/test_ip.cc
@@ -1250,7 +1250,6 @@ TEST_CASE("IPSpace Edge", "[libswoc][ipspace][edge]") {
auto spot = cspace.find(a1);
static_assert(std::is_same_v<Space::const_iterator, decltype(spot)>);
- auto &v1 = *spot;
auto const iter = cspace.find(a1);
if (auto &&[r, p] = *iter; !r.empty()) {