> 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
> we-
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/
> 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 w
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 ther