From b94bb5eb60bc8feeb5e912b6e288f0ae6b9edfe2 Mon Sep 17 00:00:00 2001
From: Guillaume Hetier <guhetier@microsoft.com>
Date: Mon, 3 Feb 2025 10:48:20 -0800
Subject: [PATCH] Remove todos

---
 src/core/recv_buffer.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/core/recv_buffer.c b/src/core/recv_buffer.c
index a41a65b939..396b66b16e 100644
--- a/src/core/recv_buffer.c
+++ b/src/core/recv_buffer.c
@@ -478,8 +478,6 @@ QuicRecvBufferCopyIntoChunks(
         }
 
         if (Chunk->Link.Flink == &RecvBuffer->Chunks) {
-            // TODO guhetier: Isn't this always true by definition, since Chunk is the last chunk?
-            //      Was this meant to update the ReadLength only if it is the first chunk instead?
             RecvBuffer->ReadLength =
                 (uint32_t)(QuicRangeGet(&RecvBuffer->WrittenRanges, 0)->Count - RecvBuffer->BaseOffset);
         }
@@ -545,7 +543,6 @@ QuicRecvBufferCopyIntoChunks(
 
         BOOLEAN IsFirstLoop = TRUE;
         do {
-            // TODO guhetier: No reason to have that in the loop, can be done once outside of it
             uint32_t ChunkWriteOffset = (ChunkOffset + RelativeOffset) % Chunk->AllocLength;
             if (!IsFirstChunk) {
                 // This RelativeOffset is already shrunk to represent the offset from beginning of the current chunk.