diff options
| author | Bruce Momjian <bruce@momjian.us> | 2008-05-13 15:44:08 +0000 |
|---|---|---|
| committer | Bruce Momjian <bruce@momjian.us> | 2008-05-13 15:44:08 +0000 |
| commit | d82a1d582cf1917d8d9807fe76bf6588091563c0 (patch) | |
| tree | 00fa8b1f3523088e76831948b35232cfc822ba2a /src | |
| parent | a61b2464fac452fbb849832a613ad5400ab7a7da (diff) | |
| download | postgresql-d82a1d582cf1917d8d9807fe76bf6588091563c0.tar.gz postgresql-d82a1d582cf1917d8d9807fe76bf6588091563c0.zip | |
This is the patch replace offnum++ by OffsetNumberNext, to be
consistent. OffsetNumberNext() has some casting that makes it useful.
Fujii Masao
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/access/heap/pruneheap.c | 4 | ||||
| -rw-r--r-- | src/backend/executor/nodeBitmapHeapscan.c | 4 | ||||
| -rw-r--r-- | src/backend/storage/page/bufpage.c | 6 |
3 files changed, 7 insertions, 7 deletions
diff --git a/src/backend/access/heap/pruneheap.c b/src/backend/access/heap/pruneheap.c index 5ee476961b3..ce3d92c6300 100644 --- a/src/backend/access/heap/pruneheap.c +++ b/src/backend/access/heap/pruneheap.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/heap/pruneheap.c,v 1.11 2008/05/12 16:06:09 alvherre Exp $ + * $PostgreSQL: pgsql/src/backend/access/heap/pruneheap.c,v 1.12 2008/05/13 15:44:08 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -792,7 +792,7 @@ heap_get_root_tuples(Page page, OffsetNumber *root_offsets) MemSet(root_offsets, 0, MaxHeapTuplesPerPage * sizeof(OffsetNumber)); maxoff = PageGetMaxOffsetNumber(page); - for (offnum = FirstOffsetNumber; offnum <= maxoff; offnum++) + for (offnum = FirstOffsetNumber; offnum <= maxoff; offnum = OffsetNumberNext(offnum)) { ItemId lp = PageGetItemId(page, offnum); HeapTupleHeader htup; diff --git a/src/backend/executor/nodeBitmapHeapscan.c b/src/backend/executor/nodeBitmapHeapscan.c index a195a049309..916340288a7 100644 --- a/src/backend/executor/nodeBitmapHeapscan.c +++ b/src/backend/executor/nodeBitmapHeapscan.c @@ -21,7 +21,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/executor/nodeBitmapHeapscan.c,v 1.27 2008/05/12 00:00:49 alvherre Exp $ + * $PostgreSQL: pgsql/src/backend/executor/nodeBitmapHeapscan.c,v 1.28 2008/05/13 15:44:08 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -302,7 +302,7 @@ bitgetpage(HeapScanDesc scan, TBMIterateResult *tbmres) OffsetNumber maxoff = PageGetMaxOffsetNumber(dp); OffsetNumber offnum; - for (offnum = FirstOffsetNumber; offnum <= maxoff; offnum++) + for (offnum = FirstOffsetNumber; offnum <= maxoff; offnum = OffsetNumberNext(offnum)) { ItemId lp; HeapTupleData loctup; diff --git a/src/backend/storage/page/bufpage.c b/src/backend/storage/page/bufpage.c index ad1c8c3ed3f..c4af3b193fc 100644 --- a/src/backend/storage/page/bufpage.c +++ b/src/backend/storage/page/bufpage.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/storage/page/bufpage.c,v 1.78 2008/02/10 20:39:08 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/storage/page/bufpage.c,v 1.79 2008/05/13 15:44:08 momjian Exp $ * *------------------------------------------------------------------------- */ @@ -533,7 +533,7 @@ PageGetHeapFreeSpace(Page page) * Since this is just a hint, we must confirm that there is * indeed a free line pointer */ - for (offnum = FirstOffsetNumber; offnum <= nline; offnum++) + for (offnum = FirstOffsetNumber; offnum <= nline; offnum = OffsetNumberNext(offnum)) { ItemId lp = PageGetItemId(page, offnum); @@ -736,7 +736,7 @@ PageIndexMultiDelete(Page page, OffsetNumber *itemnos, int nitems) totallen = 0; nused = 0; nextitm = 0; - for (offnum = 1; offnum <= nline; offnum++) + for (offnum = FirstOffsetNumber; offnum <= nline; offnum = OffsetNumberNext(offnum)) { lp = PageGetItemId(page, offnum); Assert(ItemIdHasStorage(lp)); |