Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 4c64b51

Browse files
committed
Remove dead assignment to local variable.
This should have been removed in commit 7e30c18, which split the loop into two. Only the first loop uses the 'from' variable; updating it in the second loop is bogus. It was never read after the first loop, so this was harmless and surely optimized away by the compiler, but let's be tidy. Backpatch to all supported versions. Author: Ranier Vilela Discussion: https://www.postgresql.org/message-id/CAEudQAoWq%2BAL3BnELHu7gms2GN07k-np6yLbukGaxJ1vY-zeiQ%40mail.gmail.com
1 parent 2c9b46c commit 4c64b51

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/backend/access/nbtree/nbtxlog.c

-1
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,6 @@ _bt_restore_page(Page page, char *from, int len)
7777
if (PageAddItem(page, items[i], itemsizes[i], nitems - i,
7878
false, false) == InvalidOffsetNumber)
7979
elog(PANIC, "_bt_restore_page: cannot add item to page");
80-
from += itemsz;
8180
}
8281
}
8382

0 commit comments

Comments
 (0)