pgsql: Improve the error message in COPY with HEADER option. - Mailing list pgsql-committers

From Fujii Masao
Subject pgsql: Improve the error message in COPY with HEADER option.
Date
Msg-id E1vijJR-001tu4-0e@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Improve the error message in COPY with HEADER option.

The error message reported for invalid values of the HEADER option in COPY
command previously used the term "non-negative integer", which is
discouraged by the Error Message Style Guide because it is ambiguous about
whether zero is allowed.

This commit improves the error message by replacing "non-negative integer"
there with "an integer value greater than or equal to zero" to make
the accepted values explicit.

Author: Shinya Kato <shinya11.kato@gmail.com>
Reviewed-by: Alvaro Herrera <alvherre@kurilemu.de>
Reviewed-by: Steven Niu <niushiji@gmail.com>
Reviewed-by: Fujii Masao <masao.fujii@gmail.com>
Discussion: https://postgr.es/m/CAHGQGwE86PcuPZbP=aurmW7Oo=eycF10gxjErWq4NmY-5TTX4Q@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f3da70a805f9a9dd2deada728649b2cfbeae9cb3

Modified Files
--------------
src/backend/commands/copy.c         | 17 ++++++++++-------
src/test/regress/expected/copy.out  |  2 +-
src/test/regress/expected/copy2.out |  2 +-
3 files changed, 12 insertions(+), 9 deletions(-)


pgsql-committers by date:

Previous
From: Peter Smith
Date:
Subject: Re: pgsql: tests: Add a test C++ extension module
Next
From: Thomas Munro
Date:
Subject: pgsql: jit: Add missing inline pass for LLVM >= 17.