commit 2d5d23451eb59d6fe2e558c6cd855510e1ea77e0
author QUICHE team <quiche-dev@google.com> Thu Mar 21 13:01:37 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Mar 21 14:28:03 2019 -0700
tree 3df9704a005c53620381b9f942fc8ba17aee01c3
parent bbaa8be19f38eba4539478a6e801c226a1b2c17b

gfe-relnote: (n/a) Minor tweaks to quic_interval_(set_)test. Test only.

PiperOrigin-RevId: 239654732
Change-Id: I573b20d6962af23077ac3af2d10066d783ab0cff

quic/core/quic_interval_set_test.cc

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) {