Commit 1fec8676 for tesseract
commit 1fec86762cf409bd3f623872905621639dfd176d
Author: Stefan Weil <sw@weilnetz.de>
Date: Sat Aug 15 17:28:48 2026 +0200
Fix CID 1510983 (Unused value)
Coverity reported that a value stored in a local variable is never
read before it is overwritten or the variable goes out of scope.
In PAGE_RES_IT::internal_forward, the value false assigned to
new_block is always overwritten by the assignment new_block = true
at the end of the loop body before it is read again, so the
assignment is unnecessary.
Assisted-by: OpenCode / big-pickle (opencode)
Signed-off-by: Stefan Weil <sw@weilnetz.de>
diff --git a/src/ccstruct/pageres.cpp b/src/ccstruct/pageres.cpp
index 3d07ad19..60e110b2 100644
--- a/src/ccstruct/pageres.cpp
+++ b/src/ccstruct/pageres.cpp
@@ -1635,7 +1635,6 @@ WERD_RES *PAGE_RES_IT::internal_forward(bool new_block, bool empty_ok) {
while (!block_res_it.cycled_list()) {
if (new_block) {
- new_block = false;
row_res_it.set_to_list(&block_res_it.data()->row_res_list);
row_res_it.mark_cycle_pt();
if (row_res_it.empty() && empty_ok) {