quic: address review comments

nodejs · Dec 9, 2023 · 81a31c6 · 81a31c6
1 parent 7ca1bfb
commit 81a31c6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/dataqueue/queue.cc b/src/dataqueue/queue.cc
@@ -176,10 +176,10 @@ class DataQueueImpl final : public DataQueue,
 
   void NotifyBackpressure(size_t amount) {
     if (idempotent_) return;
-    for (auto listener : backpressure_listeners_) listener->EntryRead(amount);
+    for (auto& listener : backpressure_listeners_) listener->EntryRead(amount);
   }
 
-  bool HasBackpressureListeners() const {
+  bool HasBackpressureListeners() const noexcept {
     return !backpressure_listeners_.empty();
   }
 

diff --git a/src/quic/streams.cc b/src/quic/streams.cc
@@ -804,7 +804,7 @@ BaseObjectPtr<Blob::Reader> Stream::get_reader() {
 
 void Stream::set_final_size(uint64_t final_size) {
   DCHECK_IMPLIES(state_->fin_received == 1,
-                 final_size <= stats_.Get<&Stats::final_size>());
+                 final_size <= STAT_GET(Stats, final_size));
   state_->fin_received = 1;
   STAT_SET(Stats, final_size, final_size);
 }