gfe-relnote: (n/a) Minor tweaks to quic_interval_(set_)test. Test only. PiperOrigin-RevId: 239654732 Change-Id: I573b20d6962af23077ac3af2d10066d783ab0cff
diff --git a/quic/core/quic_interval_set_test.cc b/quic/core/quic_interval_set_test.cc index f9c894d..3fb483a 100644 --- a/quic/core/quic_interval_set_test.cc +++ b/quic/core/quic_interval_set_test.cc
@@ -93,13 +93,13 @@ static bool VA_Check(const QuicIntervalSet<int>& is, int count, va_list ap) { std::vector<QuicInterval<int>> intervals(is.begin(), is.end()); if (count != static_cast<int>(intervals.size())) { - LOG(ERROR) << "Expected " << count << " intervals, got " << intervals.size() - << ": " << is; + QUIC_LOG(ERROR) << "Expected " << count << " intervals, got " + << intervals.size() << ": " << is; return false; } if (count != static_cast<int>(is.Size())) { - LOG(ERROR) << "Expected " << count << " intervals, got Size " << is.Size() - << ": " << is; + QUIC_LOG(ERROR) << "Expected " << count << " intervals, got Size " + << is.Size() << ": " << is; return false; } bool result = true; @@ -107,8 +107,8 @@ int min = va_arg(ap, int); int max = va_arg(ap, int); if (min != intervals[i].min() || max != intervals[i].max()) { - LOG(ERROR) << "Expected: [" << min << ", " << max << ") got " - << intervals[i] << " in " << is; + QUIC_LOG(ERROR) << "Expected: [" << min << ", " << max << ") got " + << intervals[i] << " in " << is; result = false; } } @@ -946,12 +946,12 @@ } struct NotOstreamable { - bool operator<(const NotOstreamable& other) const { return false; } - bool operator>(const NotOstreamable& other) const { return false; } - bool operator!=(const NotOstreamable& other) const { return false; } - bool operator>=(const NotOstreamable& other) const { return true; } - bool operator<=(const NotOstreamable& other) const { return true; } - bool operator==(const NotOstreamable& other) const { return true; } + bool operator<(const NotOstreamable&) const { return false; } + bool operator>(const NotOstreamable&) const { return false; } + bool operator!=(const NotOstreamable&) const { return false; } + bool operator>=(const NotOstreamable&) const { return true; } + bool operator<=(const NotOstreamable&) const { return true; } + bool operator==(const NotOstreamable&) const { return true; } }; TEST_F(QuicIntervalSetTest, IntervalOfTypeWithNoOstreamSupport) {
diff --git a/quic/core/quic_interval_test.cc b/quic/core/quic_interval_test.cc index b57e632..209299d 100644 --- a/quic/core/quic_interval_test.cc +++ b/quic/core/quic_interval_test.cc
@@ -438,9 +438,9 @@ } struct NotOstreamable { - bool operator<(const NotOstreamable& other) const { return false; } - bool operator>=(const NotOstreamable& other) const { return true; } - bool operator==(const NotOstreamable& other) const { return true; } + bool operator<(const NotOstreamable&) const { return false; } + bool operator>=(const NotOstreamable&) const { return true; } + bool operator==(const NotOstreamable&) const { return true; } }; TEST_F(QuicIntervalTest, IntervalOfTypeWithNoOstreamSupport) {