On 11/17/21, 4:15 PM, "Tom Lane" <tgl@sss.pgh.pa.us> wrote:
> Pushed with a little further tweaking --- mostly, I felt that
> explicitly referring to SIGINT in the API names was too
> implementation-specific, so I renamed things.
Thanks!
> As you mentioned, there are several other simple_prompt() calls
> that could usefully be improved. (I think the one in startup.c
> may be OK because we've not set up the SIGINT handler yet,
> though.) I wondered whether it's worth refactoring some more
> to have just one function that sets up the context mechanism.
I'll get started on these.
> I was also of two minds about whether to add a context option
> to pg_get_line_buf(). I stuck with your choice not to, but
> it does look a bit inconsistent.
Yeah, I figured it'd be simple enough to add that if/when it's needed.
Nathan