Incorrect comment in tableam.h regarding GetHeapamTableAmRoutine()

Started by Michael Paquierover 2 years ago3 messageshackers
Jump to latest
#1Michael Paquier
michael@paquier.xyz

Hi all,

I have noticed that GetHeapamTableAmRoutine() is listed as being a
member of tableamapi.c but it is a convenience routine located in
heapam_handler.c. Shouldn't the header be fixed with something like
the attached?

Thoughts or comments?
--
Michael

Attachments:

tableam-header-comment.patchtext/x-diff; charset=us-asciiDownload+6-0
#2Richard Guo
guofenglinux@gmail.com
In reply to: Michael Paquier (#1)
Re: Incorrect comment in tableam.h regarding GetHeapamTableAmRoutine()

On Mon, Nov 27, 2023 at 1:50 PM Michael Paquier <michael@paquier.xyz> wrote:

I have noticed that GetHeapamTableAmRoutine() is listed as being a
member of tableamapi.c but it is a convenience routine located in
heapam_handler.c. Shouldn't the header be fixed with something like
the attached?

+1. Nice catch.

Thanks
Richard

#3Michael Paquier
michael@paquier.xyz
In reply to: Richard Guo (#2)
Re: Incorrect comment in tableam.h regarding GetHeapamTableAmRoutine()

On Mon, Nov 27, 2023 at 02:14:32PM +0800, Richard Guo wrote:

+1. Nice catch.

Thanks, applied it.
--
Michael