From 6aef83c82aea43e068341d232116cc7102346fa0 Mon Sep 17 00:00:00 2001 From: Kpa-clawbot Date: Wed, 1 Apr 2026 23:25:07 -0700 Subject: [PATCH] fix: remove duplicate return statement in _cumulativeRowOffsets() (#476) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary Removes an unreachable duplicate `return offsets;` statement in the `_cumulativeRowOffsets()` function in `packets.js`. The second return was dead code found during review of PR #402. ## Changes - **`public/packets.js`**: Removed the duplicate `return offsets;` on what was line 1137 (the line immediately after the first, reachable `return offsets;`) - **`public/index.html`**: Cache buster bump ## Testing This is a dead code removal — the duplicate return was unreachable. No behavior change. No new tests needed as existing tests already cover `_cumulativeRowOffsets()` behavior. Fixes #447 Co-authored-by: you --- public/index.html | 56 +++++++++++++++++++++++------------------------ public/packets.js | 1 - 2 files changed, 28 insertions(+), 29 deletions(-) diff --git a/public/index.html b/public/index.html index 735c9925..238bce1f 100644 --- a/public/index.html +++ b/public/index.html @@ -22,9 +22,9 @@ - - - + + + @@ -85,30 +85,30 @@
- - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/public/packets.js b/public/packets.js index ef8c517a..ce85b778 100644 --- a/public/packets.js +++ b/public/packets.js @@ -1134,7 +1134,6 @@ } _cumulativeOffsetsCache = offsets; return offsets; - return offsets; } function renderVisibleRows() {