Remove dead assignment to local variable.

hlinnaka · hlinnaka · commit 4df5f6c26b2d · 2021-07-12T11:14:06.000+03:00
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
diff --git 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;
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,6 @@ _bt_restore_page(Page page, char *from, int len)`
`74`	`74`	`if (PageAddItem(page, items[i], itemsizes[i], nitems - i,`
`75`	`75`	`false, false) == InvalidOffsetNumber)`
`76`	`76`	`elog(PANIC, "_bt_restore_page: cannot add item to page");`
`77`		`- from += itemsz;`
`78`	`77`	`}`
`79`	`78`	`}`
`80`	`79`