From: | Doug Doole <DougDoole(at)gmail(dot)com> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Cc: | Oliver Ford <ojford(at)gmail(dot)com> |
Subject: | Re: Add Roman numeral conversion to to_number |
Date: | 2017-09-15 01:00:27 |
Message-ID: | 20170915010027.1354.42875.pgcf@coridan.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: not tested
Documentation: not tested
Code looks fine, but one niggly complaint at line 146 of the patch file ("while (*cp) {"). A K&R style brace slipped in, which doesn't match the formatting of the file.
Given that this is providing new formatting options, there should be new tests added that validate the options and error handling.
There's also the "do we want this?" debate from the discussion thread that still needs to be resolved. (I don't have an opinion either way.)
I'm sending this back to the author to address the first two issues.
The new status of this patch is: Waiting on Author
From | Date | Subject | |
---|---|---|---|
Next Message | Tomas Vondra | 2017-09-15 01:07:25 | Re: PATCH : Generational memory allocator (was PATCH: two slab-like memory allocators) |
Previous Message | Tsunakawa, Takayuki | 2017-09-15 00:43:59 | Re: Process startup infrastructure is a mess |