At Wed, 30 Aug 2023 10:21:26 +0900 (JST), Kyotaro Horiguchi <horikyota....@gmail.com> wrote in > Recently \watch got the following help message. > > > \watch [[i=]SEC] [c=N] [m=MIN] > > execute query every SEC seconds, up to N times > > stop if less than MIN rows are returned > > The "m=MIN" can be a bit misleading. It may look like it's about > interval or counts, but it actually refers to the row number, which is > not spelled out in the line. > > Would it make sense to change it to MINROWS? There's enough room in > the line for that change and the doc already uses min_rows.
Mmm. I noticed the continuation lines are indented too much, probably because of the backslash escape in the main line. The attached includes the fix for that. regards. -- Kyotaro Horiguchi NTT Open Source Software Center
diff --git a/src/bin/psql/help.c b/src/bin/psql/help.c index 38c165a627..2da79a75f5 100644 --- a/src/bin/psql/help.c +++ b/src/bin/psql/help.c @@ -200,9 +200,9 @@ slashUsage(unsigned short int pager) HELP0(" \\gset [PREFIX] execute query and store result in psql variables\n"); HELP0(" \\gx [(OPTIONS)] [FILE] as \\g, but forces expanded output mode\n"); HELP0(" \\q quit psql\n"); - HELP0(" \\watch [[i=]SEC] [c=N] [m=MIN]\n"); - HELP0(" execute query every SEC seconds, up to N times\n"); - HELP0(" stop if less than MIN rows are returned\n"); + HELP0(" \\watch [[i=]SEC] [c=N] [m=MINROWS]\n"); + HELP0(" execute query every SEC seconds, up to N times\n"); + HELP0(" stop if less than MINROWS rows are returned\n"); HELP0("\n"); HELP0("Help\n");