From: | btsugieyuusuke <btsugieyuusuke(at)oss(dot)nttdata(dot)com> |
---|---|
To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pg_walsummary, Character-not-present-in-option |
Date: | 2024-09-30 10:40:30 |
Message-ID: | 2f64082978c73e2f3c84b4a4ea533296@oss.nttdata.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I forgot to attach the patch file, so I'm attaching it in reply.
2024-09-30 15:02 に btsugieyuusuke さんは書きました:
> Hi hackers,
> I found probably something to fix in pg_walsummary.
>
> pg_walsummary specifies “f:iqw:” as the third argument of
> getopt_long().
>
>> /* process command-line options */
>> while ((c = getopt_long(argc, argv, "f:iqw:",
>> long_options, &optindex)) != -1)
>
> However, only i and q are valid options.
>
>> switch (c)
>> {
>> case 'i':
>> break;
>> case 'q':
>> opt.quiet = true;
>> break;
>> default:
>> /* getopt_long already emitted a complaint */
>> pg_log_error_hint("Try \"%s --help\" for more information.",
>> progname);
>> exit(1);
>> }
>
> Therefore, shouldn't “f:” and “w:” be removed?
>
> Best regards,
> Yusuke Sugie
Attachment | Content-Type | Size |
---|---|---|
0001-Character-not-present-in-option.patch | text/x-diff | 830 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Lakhin | 2024-09-30 11:00:00 | Re: msys inet_pton strangeness |
Previous Message | Yugo Nagata | 2024-09-30 10:38:35 | Re: ACL_MAINTAIN, Lack of comment content |