From c76ed8151305d724104cd355f15ea982d08e3bc4 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 6 Feb 2007 14:55:11 +0000 Subject: [PATCH] Remove some dead code, per Heikki. --- src/backend/access/nbtree/nbtinsert.c | 29 +++++++++++---------------- 1 file changed, 12 insertions(+), 17 deletions(-) diff --git a/src/backend/access/nbtree/nbtinsert.c b/src/backend/access/nbtree/nbtinsert.c index 0a74148693..a743da1cc4 100644 --- a/src/backend/access/nbtree/nbtinsert.c +++ b/src/backend/access/nbtree/nbtinsert.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/nbtree/nbtinsert.c,v 1.148 2007/01/27 20:53:30 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/access/nbtree/nbtinsert.c,v 1.149 2007/02/06 14:55:11 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -855,22 +855,17 @@ _bt_split(Relation rel, Buffer buf, OffsetNumber firstright, /* cope with possibility that newitem goes at the end */ if (i <= newitemoff) { - if (newitemonleft) - { - _bt_pgaddtup(rel, leftpage, newitemsz, newitem, leftoff, - "left sibling"); - itup_off = leftoff; - itup_blkno = BufferGetBlockNumber(buf); - leftoff = OffsetNumberNext(leftoff); - } - else - { - _bt_pgaddtup(rel, rightpage, newitemsz, newitem, rightoff, - "right sibling"); - itup_off = rightoff; - itup_blkno = BufferGetBlockNumber(rbuf); - rightoff = OffsetNumberNext(rightoff); - } + /* + * Can't have newitemonleft here; that would imply we were told to put + * *everything* on the left page, which cannot fit (if it could, we'd + * not be splitting the page). + */ + Assert(!newitemonleft); + _bt_pgaddtup(rel, rightpage, newitemsz, newitem, rightoff, + "right sibling"); + itup_off = rightoff; + itup_blkno = BufferGetBlockNumber(rbuf); + rightoff = OffsetNumberNext(rightoff); } /* -- 2.40.0