Re: [PATCH] Cleanup of GUC units code

Поиск
Список
Период
Сортировка
От Marko Kreen
Тема Re: [PATCH] Cleanup of GUC units code
Дата
Msg-id e51f66da0809010637r182327acx7c637c4cd9d3e4e5@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Cleanup of GUC units code  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [PATCH] Cleanup of GUC units code
Список pgsql-hackers
On 9/1/08, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Marko Kreen <markokr@gmail.com> writes:
>  > - In attempt to preserve maximum range of values for INT64_IS_BUSTED
>  >   systems, the code is written rather non-obvious way.
>
> I do not personally object a bit to making the units comparisons
>  case-insensitive (I think it's mainly Peter who wants to be strict
>  about it).  I don't think there are any other good ideas in this
>  patch, however, and exposing ourselves to intermediate overflows in
>  the name of simplicity is definitely not one.

For all practical purposes, the overflow is insignificant when int64
works.  I'll look if I can avoid it on INT64_IS_BUSTED case.

In the meantime, here is simple patch for case-insensivity.

--
marko

Вложения

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

Предыдущее
От: "Marko Kreen"
Дата:
Сообщение: Re: [PATCH] Make gram.y use palloc/pfree for memory management
Следующее
От: Stephen Frost
Дата:
Сообщение: Re: Extending grant insert on tables to sequences