]> granicus.if.org Git - postgresql/commit
Update buffile.h/.c comments for removal of non-temp option.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 25 Nov 2017 18:19:43 +0000 (13:19 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 25 Nov 2017 18:19:43 +0000 (13:19 -0500)
commitab97aaac8f058f2e16ef08655d185db20bc241d3
treeadc8c2448eebaa8d79f4aa32d011514e30494ce6
parentdf3a66e282b66971b3c08896176b95f745466a64
Update buffile.h/.c comments for removal of non-temp option.

Commit 11e264517 removed BufFile's isTemp flag, thereby eliminating
the possibility of resurrecting BufFileCreate().  But it left that
function in place, as well as a bunch of comments describing how things
worked for the non-temp-file case.  At best, that's now a source of
confusion.  So remove the long-since-commented-out function and change
relevant comments.

I (tgl) wanted to rename BufFileCreateTemp() to BufFileCreate(), but
that seems not to be the consensus position, so leave it as-is.

In passing, fix commit f0828b2fc's failure to update BufFileSeek's
comment to match the change of its argument type from long to off_t.
(I think that might actually have been intentional at the time, but
now that 64-bit off_t is nearly universal, it looks anachronistic.)

Thomas Munro and Tom Lane

Discussion: https://postgr.es/m/E1eFVyl-0008J1-RO@gemulon.postgresql.org
src/backend/storage/file/buffile.c
src/include/storage/buffile.h