nbtree: Rename nbtinsert.c variables for consistency.
authorPeter Geoghegan <pg@bowt.ie>
Tue, 17 Nov 2020 17:01:14 +0000 (09:01 -0800)
committerPeter Geoghegan <pg@bowt.ie>
Tue, 17 Nov 2020 17:01:14 +0000 (09:01 -0800)
commita034f8b60c6e9afd75f42dc420a76cb9d3875fcb
treecc316bb69ea65c4f24e5012a020aa7ba4a3b5ac1
parent9653f24ad8307f393de51e0a64d9b10a49efa6e3
nbtree: Rename nbtinsert.c variables for consistency.

Stop naming special area/opaque pointer variables 'lpageop' in contexts
where it doesn't make sense.  This is a holdover from a time when logic
that performs tasks that are now spread across _bt_insertonpg(),
_bt_findinsertloc(), and _bt_split() was more centralized.  'lpageop'
denotes "left page", which doesn't make sense outside of contexts in
which there isn't also a right page.

Also acquire page flag variables up front within _bt_insertonpg().  This
makes it closer to _bt_split() following refactoring commit bc3087b626d.
This allows the page split and retail insert paths to both make use of
the same variables.
src/backend/access/nbtree/nbtinsert.c