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");

Reply via email to