Fix is_digit labeling of to_timestamp's FFn format codes.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 7 Dec 2024 18:12:32 +0000 (13:12 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 7 Dec 2024 18:12:32 +0000 (13:12 -0500)
commit26c233b8b8f536f4ed252f9145c2ee365bb04b99
treedf810491ec0b4bb1b95c09806e711c7e13a4fdcf
parentad485171c703915b09a9f498fad69e89f75c9568
Fix is_digit labeling of to_timestamp's FFn format codes.

These format codes produce or consume strings of digits, so they
should be labeled with is_digit = true, but they were not.
This has effect in only one place, where is_next_separator()
is checked to see if the preceding format code should slurp up
all the available digits.  Thus, with a format such as '...SSFF3'
with remaining input '12345', the 'SS' code would consume all
five digits (and then complain about seconds being out of range)
when it should eat only two digits.

Per report from Nick Davies.  This bug goes back to d589f9446
where the FFn codes were introduced, so back-patch to v13.

Discussion: https://postgr.es/m/AM8PR08MB6356AC979252CFEA78B56678B6312@AM8PR08MB6356.eurprd08.prod.outlook.com
src/backend/utils/adt/formatting.c
src/test/regress/expected/horology.out
src/test/regress/sql/horology.sql