Re: Possibly hard-to-read message

On Sun, Apr 6, 2025 at 2:52 PM Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:

>
> Looking at the other variables they tend to use "if set, then" so we should
> probably stick to that for this as well? Something like the below perhaps?
>
> HELP0(" WATCH_INTERVAL\n"
> - " number of seconds \\watch by default waits between
> executing the query buffer\n");
> + " if set to an interval value, overrides the default
> two second \\watch interval\n");
>
>
Using "interval" here could cause minor confusion since it is a valid core
data type but not accepted here.

psql --set=WATCH_INTERVAL='10m' postgres
psql: error: invalid value "10m" for "WATCH_INTERVAL"

select '10m'::interval;
interval
----------
00:10:00

How about:

+ " if set to a number, overrides the default two second \\watch
interval\n"

I do like the consistency of "if set" even though most of the others are
booleans.

David J.