Wrong function name in pgstatfuncs.c

Started by Kyotaro Horiguchiover 2 years ago2 messageshackers
Jump to latest
#1Kyotaro Horiguchi
horikyota.ntt@gmail.com

Since pgstatfuncs.c was refactored, the comments for synthesized
function names are significant to find the function body.

I happened to find a misspelling among the function name
comments. "pg_stat_get_mods_since_analyze" should be
"pg_stat_get_mod_since_analyze".

Upon checking the file using a rudimentary script, I found no other
similar mistakes in the same file.

(FWIW, I also feel that these macros might be going a bit too far by
synthesizing even the function names.)

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

Attachments:

fix_funcname_in_comment.patchtext/x-patch; charset=us-asciiDownload+1-1
#2Daniel Gustafsson
daniel@yesql.se
In reply to: Kyotaro Horiguchi (#1)
Re: Wrong function name in pgstatfuncs.c

On 1 Nov 2023, at 09:23, Kyotaro Horiguchi <horikyota.ntt@gmail.com> wrote:

Since pgstatfuncs.c was refactored, the comments for synthesized
function names are significant to find the function body.

I happened to find a misspelling among the function name
comments. "pg_stat_get_mods_since_analyze" should be
"pg_stat_get_mod_since_analyze".

Upon checking the file using a rudimentary script, I found no other
similar mistakes in the same file.

Nice catch, that's indeed a tiny typo, will fix.

--
Daniel Gustafsson