| От | Neil Conway |
|---|---|
| Тема | Re: [COMMITTERS] pgsql: Add comments about why errno is |
| Дата | |
| Msg-id | 1133475547.16064.3.camel@localhost.localdomain обсуждение |
| Ответ на | Re: [COMMITTERS] pgsql: Add comments about why errno is set (Bruce Momjian <pgman@candle.pha.pa.us>) |
| Ответы |
Re: [COMMITTERS] pgsql: Add comments about why errno is
|
| Список | pgsql-hackers |
On Thu, 2005-12-01 at 16:38 -0500, Bruce Momjian wrote:
> Maybe it should be:
>
> errno = 0; /* Allow unconditional errno check */
I think any solution that involves adding more duplication at each
strtol() callsite is not great ("Don't Repeat Yourself"). I'd still like
to see this refactored into a separate function, as I suggested on
-patches. If people would like to see a detailed explanation of the
interaction between strtol() and errno, a header comment to pg_strtol()
seems a good place to put it. IMO that is better than copying and
pasting a cryptic one-line comment to each and every callsite of
strtol().
-Neil
В списке pgsql-hackers по дате отправления:
Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера