Re: [PATCH] Add roman support for to_number function
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Maciek Sakrejda <m.sakrejda@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org, Hunaid Sohail <hunaidpgml@gmail.com>
Date: 2024-09-08T21:44:20Z
Lists: pgsql-hackers
I wrote: > * Further to Aleksander's point about lack of test coverage for > the to_char direction, I see from > https://coverage.postgresql.org/src/backend/utils/adt/formatting.c.gcov.html > that almost none of the existing roman-number code paths are covered > today. While it's not strictly within the charter of this patch > to improve that, maybe we should try to do so --- at the very > least it'd raise formatting.c's coverage score a few notches. For the archives' sake: I created a patch and a separate discussion thread [1] for that. regards, tom lane [1] https://www.postgresql.org/message-id/flat/2956175.1725831136@sss.pgh.pa.us
Commits
-
Support RN (roman-numeral format) in to_number().
- 172e6b3adb2e 18.0 landed