| From: | Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp> |
|---|---|
| To: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Rename setup_cancel_handler in pg_dump |
| Date: | 2024-02-08 01:07:49 |
| Message-ID: | 20240208100749.5fd43140906c8f973f294b9a@sraoss.co.jp |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, 7 Feb 2024 22:59:48 +0100
Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
> > On 1 Feb 2024, at 02:21, Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp> wrote:
> > On Tue, 30 Jan 2024 13:44:28 +0100
> > Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
> >> -setup_cancel_handler(void)
> >> +pg_dump_setup_cancel_handler(void)
> >>
> >> We don't have any other functions prefixed with pg_dump_, based on the naming
> >> of the surrounding code in the file I wonder if set_cancel_handler is a more
> >> appropriate name?
> >
> > Agreed. Here is a updated patch.
>
> Sleeping on it I still think this is a good idea, and hearing no objections I
> went ahead with this. Thanks for the patch!
Thank you!
Regards,
Yugo Nagata
>
> --
> Daniel Gustafsson
>
--
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2024-02-08 01:27:59 | Re: cfbot is failing all tests on FreeBSD/Meson builds |
| Previous Message | Noah Misch | 2024-02-08 00:55:41 | Re: Should we remove -Wdeclaration-after-statement? |