On Thu, Nov 2, 2023 at 10:20 PM Daniel Gustafsson <dan...@yesql.se> wrote: > > On 2 Nov 2023, at 13:40, Etsuro Fujita <etsuro.fuj...@gmail.com> wrote: > > Attached is a small patch for that: s/heapam_visibility.c/snapmgr.c/. > > No objections to the patch, the change is correct. However, with git grep and > ctags and other ways of discovery I wonder if we're not better off avoiding > such references to filenames which are prone to going stale (and do from time > to time).
Agreed. As XidInMVCCSnapshot() is an extern function, such a tool would allow the reader to easily find the source file that contains the definition of that function. Thanks for the comment! Best regards, Etsuro Fujita
