From 55e4ddcd8c9f462f9409d2a7ab8a7929d839359f Mon Sep 17 00:00:00 2001 From: Yi Kong Date: Thu, 16 May 2024 02:48:54 +0900 Subject: Backport change to fix build on ToT Clang This backports upstream change: https://quiche.googlesource.com/quiche/+/c1af894e0f5c4f732a983e7c93227854e203570e Fix QuicIntervalDeque::Iterator::operator+= The code was referring to a member which didn't exist. This was uncovered by a recent Clang change [1] which made the compiler lookup these names at template parse time rather than instantiation time. 1. https://github.com/llvm/llvm-project/pull/90152 PiperOrigin-RevId: 630952202 Test: presubmit Change-Id: I82121b22326dd0a5b0db65463b11f174b6aa81fe --- net/third_party/quiche/src/quiche/quic/core/quic_interval_deque.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/third_party/quiche/src/quiche/quic/core/quic_interval_deque.h b/net/third_party/quiche/src/quiche/quic/core/quic_interval_deque.h index 0aa4543ff..5ca3cc98a 100644 --- a/net/third_party/quiche/src/quiche/quic/core/quic_interval_deque.h +++ b/net/third_party/quiche/src/quiche/quic/core/quic_interval_deque.h @@ -198,12 +198,12 @@ class QUICHE_NO_EXPORT QuicIntervalDeque { Iterator operator+(difference_type amount) const { Iterator copy = *this; copy.index_ += amount; - QUICHE_DCHECK(copy.index_ < copy.deque_->size()); + QUICHE_DCHECK(copy.index_ < copy.deque_->Size()); return copy; } Iterator& operator+=(difference_type amount) { index_ += amount; - QUICHE_DCHECK(index_ < deque_->size()); + QUICHE_DCHECK(index_ < deque_->Size()); return *this; } difference_type operator-(const Iterator& rhs) const { -- cgit v1.2.3