Hi Bruce I saw this commit; commit ad23adc5a169b114f9ff325932cbf2ce1c5e69c1 |Author: Bruce Momjian <br...@momjian.us> |Date: Tue Apr 30 14:06:57 2019 -0400 | | doc: improve PG 12 to_timestamp()/to_date() wording
which cleans up language added at cf984672. Can I suggest this additional change, which is updated and extracted from my larger set of documentation fixes. Justin diff --git a/doc/src/sgml/func.sgml b/doc/src/sgml/func.sgml index 96fafdd..b420585 100644 --- a/doc/src/sgml/func.sgml +++ b/doc/src/sgml/func.sgml @@ -6400,20 +6400,20 @@ SELECT regexp_match('abc01234xyz', '(?:(.*?)(\d+)(.*)){1,1}'); </para> <para> If <literal>FX</literal> is specified, a separator in the template string - matches exactly one character in input string. Notice we don't insist the - input string character be the same as the template string separator. + matches exactly one character in the input string. But note that the + input string character is not required to be the same as the separator from the template string. For example, <literal>to_timestamp('2000/JUN', 'FXYYYY MON')</literal> works, but <literal>to_timestamp('2000/JUN', 'FXYYYY MON')</literal> - returns an error because the second template string space is consumed - by the letter <literal>J</literal> in the input string. + returns an error because the second space in the template string consumes + the letter <literal>M</literal> from the input string. </para> </listitem> <listitem> <para> A <literal>TZH</literal> template pattern can match a signed number. - Without the <literal>FX</literal> option, it can lead to ambiguity in - interpretation of the minus sign, which can also be interpreted as a separator. + Without the <literal>FX</literal> option, minus signs may be ambiguous, + and could be interpreted as a separator. This ambiguity is resolved as follows: If the number of separators before <literal>TZH</literal> in the template string is less than the number of separators before the minus sign in the input string, the minus sign
>From 0d304935aa5b83e56127b5a63aa3d180b4c42e16 Mon Sep 17 00:00:00 2001 From: Justin Pryzby <pryz...@telsasoft.com> Date: Sat, 30 Mar 2019 03:42:35 -0500 Subject: [PATCH v3] Clean up language in cf984672: Improve behavior of to_timestamp()/to_date() functions --- doc/src/sgml/func.sgml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/doc/src/sgml/func.sgml b/doc/src/sgml/func.sgml index 96fafdd..b420585 100644 --- a/doc/src/sgml/func.sgml +++ b/doc/src/sgml/func.sgml @@ -6400,20 +6400,20 @@ SELECT regexp_match('abc01234xyz', '(?:(.*?)(\d+)(.*)){1,1}'); </para> <para> If <literal>FX</literal> is specified, a separator in the template string - matches exactly one character in input string. Notice we don't insist the - input string character be the same as the template string separator. + matches exactly one character in the input string. But note that the + input string character is not required to be the same as the separator from the template string. For example, <literal>to_timestamp('2000/JUN', 'FXYYYY MON')</literal> works, but <literal>to_timestamp('2000/JUN', 'FXYYYY MON')</literal> - returns an error because the second template string space is consumed - by the letter <literal>J</literal> in the input string. + returns an error because the second space in the template string consumes + the letter <literal>M</literal> from the input string. </para> </listitem> <listitem> <para> A <literal>TZH</literal> template pattern can match a signed number. - Without the <literal>FX</literal> option, it can lead to ambiguity in - interpretation of the minus sign, which can also be interpreted as a separator. + Without the <literal>FX</literal> option, minus signs may be ambiguous, + and could be interpreted as a separator. This ambiguity is resolved as follows: If the number of separators before <literal>TZH</literal> in the template string is less than the number of separators before the minus sign in the input string, the minus sign -- 2.7.4