Hackers, I stumbled across an initialization of a BlockNumber with InvalidBuffer, which seems strange to me, as the values for "invalid" of the two types are different, see attached patch.
In case the 'stack' argument passed to that function is not NULL, the variable in question gets overridden immediately, in which case it certainly doesn't matter. I don't know nor did I check whether or not it can ever be NULL. So this might not be a real issue at all. Regards Markus Wanner
# InvalidBlockNumber is -1 (or rather 0xFFFFFFFF), while # the currently used InvalidBuffer is 0, which is a valid # BlockNumber. ============================================================ *** src/backend/access/gin/ginbtree.c 2d3e63387737b4034fc25ca3cb128d9ac57f4f01 --- src/backend/access/gin/ginbtree.c 67351e1b6541b25ab3c8e8dc7a57487c2422e124 *************** ginInsertValue(GinBtree btree, GinBtreeS *** 276,282 **** ginInsertValue(GinBtree btree, GinBtreeStack *stack, GinStatsData *buildStats) { GinBtreeStack *parent = stack; ! BlockNumber rootBlkno = InvalidBuffer; Page page, rpage, lpage; --- 276,282 ---- ginInsertValue(GinBtree btree, GinBtreeStack *stack, GinStatsData *buildStats) { GinBtreeStack *parent = stack; ! BlockNumber rootBlkno = InvalidBlockNumber; Page page, rpage, lpage;
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers