diff options
author | Michael Paquier <michael@paquier.xyz> | 2023-11-17 10:46:50 +0900 |
---|---|---|
committer | Michael Paquier <michael@paquier.xyz> | 2023-11-17 10:46:50 +0900 |
commit | c99c7a4871187a8bcf717147ff08e05a49e678d5 (patch) | |
tree | d4215dfd96145c276f0afd3c90d94ca53e9ee7de | |
parent | 3af0d17acef7538c5605afa5dd7929cfd7e38d9e (diff) | |
download | postgresql-c99c7a4871187a8bcf717147ff08e05a49e678d5.tar.gz postgresql-c99c7a4871187a8bcf717147ff08e05a49e678d5.zip |
Remove NOT_USED BufFileTellBlock() from buffile.c
This routine has been marked as NOT_USED since 20ad43b576d9 from 2000,
and a patch is planned to switch the logtape/tuplestore APIs to rely on
int64 rather than long for the block nunbers, which is more portable.
Keeping it is more confusing than anything at this stage, so let's get
rid of it entirely.
Thanks for Heikki Linnakangas for the poke on this one.
Discussion: https://postgr.es/m/5047be8c-7ee6-4dd5-af76-6c916c3103b4@iki.fi
-rw-r--r-- | src/backend/storage/file/buffile.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/src/backend/storage/file/buffile.c b/src/backend/storage/file/buffile.c index 41ab64100e3..d3f0f588580 100644 --- a/src/backend/storage/file/buffile.c +++ b/src/backend/storage/file/buffile.c @@ -856,24 +856,6 @@ BufFileSeekBlock(BufFile *file, long blknum) SEEK_SET); } -#ifdef NOT_USED -/* - * BufFileTellBlock --- block-oriented tell - * - * Any fractional part of a block in the current seek position is ignored. - */ -long -BufFileTellBlock(BufFile *file) -{ - long blknum; - - blknum = (file->curOffset + file->pos) / BLCKSZ; - blknum += file->curFile * BUFFILE_SEG_SIZE; - return blknum; -} - -#endif - /* * Return the current fileset based BufFile size. * |