Fix incorrect function reference BufFileOpenShared in comment.

Started by Zhang Mingli10 months ago5 messageshackers
Jump to latest
#1Zhang Mingli
zmlpostgres@gmail.com

Hi,

The comment incorrectly cited the defunct function BufFileOpenShared()
which is replaced in commit dcac5e7ac157964f71f15d81c7429130c69c3f9b.

Update the reference to the current function: BufFileOpenFileSet().

--
Zhang Mingli
HashData

Attachments:

v0-0001-Fix-incorrect-function-reference-in-comment.patchapplication/octet-streamDownload+1-2
#2wenhui qiu
qiuwenhuifx@gmail.com
In reply to: Zhang Mingli (#1)
Re: Fix incorrect function reference BufFileOpenShared in comment.

Thank you for your path ,The path LGTM

On Wed, 1 Oct 2025 at 16:15, Zhang Mingli <zmlpostgres@gmail.com> wrote:

Show quoted text

Hi,

The comment incorrectly cited the defunct function BufFileOpenShared()
which is replaced in commit dcac5e7ac157964f71f15d81c7429130c69c3f9b.

Update the reference to the current function: BufFileOpenFileSet().

--
Zhang Mingli
HashData

#3Richard Guo
guofenglinux@gmail.com
In reply to: Zhang Mingli (#1)
Re: Fix incorrect function reference BufFileOpenShared in comment.

On Wed, Oct 1, 2025 at 5:15 PM Zhang Mingli <zmlpostgres@gmail.com> wrote:

The comment incorrectly cited the defunct function BufFileOpenShared()
which is replaced in commit dcac5e7ac157964f71f15d81c7429130c69c3f9b.

Update the reference to the current function: BufFileOpenFileSet().

Nice catch. Running `git grep 'BufFile.*Shared'` confirms that this
is the only place referencing the outdated BufFileXXXShared()
functions. I'll push this patch barring any objections.

- Richard

#4Richard Guo
guofenglinux@gmail.com
In reply to: Richard Guo (#3)
Re: Fix incorrect function reference BufFileOpenShared in comment.

On Wed, Oct 1, 2025 at 6:29 PM Richard Guo <guofenglinux@gmail.com> wrote:

On Wed, Oct 1, 2025 at 5:15 PM Zhang Mingli <zmlpostgres@gmail.com> wrote:

The comment incorrectly cited the defunct function BufFileOpenShared()
which is replaced in commit dcac5e7ac157964f71f15d81c7429130c69c3f9b.

Update the reference to the current function: BufFileOpenFileSet().

Nice catch. Running `git grep 'BufFile.*Shared'` confirms that this
is the only place referencing the outdated BufFileXXXShared()
functions. I'll push this patch barring any objections.

Pushed.

- Richard

#5Zhang Mingli
zmlpostgres@gmail.com
In reply to: Richard Guo (#4)
Re: Fix incorrect function reference BufFileOpenShared in comment.

Thanks.

Richard Guo <guofenglinux@gmail.com>于2025年10月3日 周五15:41写道:

Show quoted text

On Wed, Oct 1, 2025 at 6:29 PM Richard Guo <guofenglinux@gmail.com> wrote:

On Wed, Oct 1, 2025 at 5:15 PM Zhang Mingli <zmlpostgres@gmail.com>

wrote:

The comment incorrectly cited the defunct function BufFileOpenShared()
which is replaced in commit dcac5e7ac157964f71f15d81c7429130c69c3f9b.

Update the reference to the current function: BufFileOpenFileSet().

Nice catch. Running `git grep 'BufFile.*Shared'` confirms that this
is the only place referencing the outdated BufFileXXXShared()
functions. I'll push this patch barring any objections.

Pushed.

- Richard