Editing errors in the comments of tableam.h and heapam.c

Started by Hironobu SUZUKIalmost 6 years ago3 messageshackers
Jump to latest
#1Hironobu SUZUKI
hironobu@interdb.jp

Hi,

Some comments in tableam.h and heapam.c contain three old function names
although these have been renamed by this commit
73b8c3bd2889fed986044e15aefd0911f96ccdd3.

Old: table_insert, table_fetch_row_version, table_get_latest_tid.

New: table_tuple_insert, table_tuple_fetch_row_version,
table_tuple_get_latest_tid.

I think these are editing errors. PG 12 also has the same errors.

Best regards,

Attachments:

editing-errors-correction-in-tableam-h-and-heapam-c.patchtext/plain; charset=UTF-8; name=editing-errors-correction-in-tableam-h-and-heapam-c.patch; x-mac-creator=0; x-mac-type=0Download+9-9
#2Michael Paquier
michael@paquier.xyz
In reply to: Hironobu SUZUKI (#1)
Re: Editing errors in the comments of tableam.h and heapam.c

On Mon, Jul 13, 2020 at 02:25:39PM +0200, Hironobu SUZUKI wrote:

Some comments in tableam.h and heapam.c contain three old function names
although these have been renamed by this commit
73b8c3bd2889fed986044e15aefd0911f96ccdd3.

Old: table_insert, table_fetch_row_version, table_get_latest_tid.

New: table_tuple_insert, table_tuple_fetch_row_version,
table_tuple_get_latest_tid.

Thanks. That looks right.

I think these are editing errors. PG 12 also has the same errors.

Yeah. We also recommend to look at tableam.h in the docs, so while
usually I just bother fixing comments on HEAD, it would be better to
back-patch this one.
--
Michael

#3Michael Paquier
michael@paquier.xyz
In reply to: Michael Paquier (#2)
Re: Editing errors in the comments of tableam.h and heapam.c

On Mon, Jul 13, 2020 at 09:52:12PM +0900, Michael Paquier wrote:

Yeah. We also recommend to look at tableam.h in the docs, so while
usually I just bother fixing comments on HEAD, it would be better to
back-patch this one.

Committed and back-patched down to 12. Thanks, Suzuki-san.
--
Michael