Re: Minor pg_amcheck fixes spotted while reading code

2021-08-23 Thread Daniel Gustafsson
> On 21 Aug 2021, at 02:43, Michael Paquier wrote: > > On Fri, Aug 20, 2021 at 11:42:09AM -0700, Mark Dilger wrote: >> These look correct. > > static void > -help(const char *progname) > +help(const char *program_name) > These were discussed not long ago, and I recall that they were in the >

Re: Minor pg_amcheck fixes spotted while reading code

2021-08-20 Thread Michael Paquier
On Fri, Aug 20, 2021 at 11:42:09AM -0700, Mark Dilger wrote: > These look correct. static void -help(const char *progname) +help(const char *program_name) These were discussed not long ago, and I recall that they were in the we-don't-care category. Note for example all the tools of src/scripts/

Re: Minor pg_amcheck fixes spotted while reading code

2021-08-20 Thread Mark Dilger
> On Aug 20, 2021, at 11:19 AM, Daniel Gustafsson wrote: > > When perusing the pg_amcheck code the other day for the recently pushed fixes > I > found two small things that seem worth fixing: The help function progname > parameter shadows the global variable of the same name, which no doubt

Minor pg_amcheck fixes spotted while reading code

2021-08-20 Thread Daniel Gustafsson
When perusing the pg_amcheck code the other day for the recently pushed fixes I found two small things that seem worth fixing: The help function progname parameter shadows the global variable of the same name, which no doubt will make static analyzers complain so we might as well fix that; and