On Thu, 4 Jul 2024 at 04:19, Tom Lane <t...@sss.pgh.pa.us> wrote: > - * Return the current fileset based BufFile size. > + * Returns the size if the given BufFile in bytes. > > "Returns the size of", no doubt?
Yes, thanks. > A shade less nit-pickily, I wonder if "size" is sufficient. > It's not really obvious that this means the amount of data > in the file, rather than say sizeof(BufFile). How about > > + * Returns the amount of data in the given BufFile, in bytes. I've used that. Thank you. > LGTM other than that point. Pushed. David
