postgres.git / summary / log / commit / refs

commit    4df5f6c26b2d5c89f1bc71eaee34e354974e8b97
Author:   Heikki Linnakangas <heikki.linnakangas@iki.fi>
Date:     Mon Jul 12 08:13:33 2021 +0000
First released in: 15.0, 9.6.23, 10.18, 11.13, 12.8, 13.4, 14.0

    Remove dead assignment to local variable.
    
    This should have been removed in commit 7e30c186da, 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


 src/backend/access/nbtree/nbtxlog.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/backend/access/nbtree/nbtxlog.c b/src/backend/access/nbtree/nbtxlog.c
index dd5315c1aad..02c61342aa3 100644
--- a/src/backend/access/nbtree/nbtxlog.c
+++ b/src/backend/access/nbtree/nbtxlog.c
@@ -74,7 +74,6 @@ _bt_restore_page(Page page, char *from, int len)
 		if (PageAddItem(page, items[i], itemsizes[i], nitems - i,
 						false, false) == InvalidOffsetNumber)
 			elog(PANIC, "_bt_restore_page: cannot add item to page");
-		from += itemsz;
 	}
 }
 


[parent: 92340ba5a795]