Re: Change atoi to strtol in same place

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема Re: Change atoi to strtol in same place
Дата
Msg-id d9787691-7d5d-e3e0-f527-1bb152945c7c@2ndquadrant.com
обсуждение исходный текст
Ответ на Re: Change atoi to strtol in same place  (Joe Nelson <joe@begriffs.com>)
Ответы Re: refactoring - standardize integer parsing in front-end utilities
Список pgsql-hackers
On 2019-12-06 18:43, Joe Nelson wrote:
> I see this patch has been moved to the next commitfest. What's the next
> step, does it need another review?

I think you need to promote your patch better.  The thread subject and 
the commit fest entry title are somewhat nonsensical and no longer match 
what the patch actually does.  The patch contains no commit message and 
no documentation or test changes, so it's not easy to make out what it's 
supposed to do or verify that it does so correctly.  A reviewer would 
have to take this patch on faith or manually test every single command 
line option to see what it does.  Moreover, a lot of this error message 
tweaking is opinion-based, so it's more burdensome to do an objective 
review.  This patch is competing for attention against more than 200 
other patches that have more going for them in these aspects.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Fabien COELHO
Дата:
Сообщение: Re: pgbench - use pg logging capabilities
Следующее
От: Peter Eisentraut
Дата:
Сообщение: Re: Remove libpq.rc, use win32ver.rc for libpq