Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint
Дата
Msg-id 553198.1635453246@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint  (Andres Freund <andres@anarazel.de>)
Список pgsql-hackers
Andres Freund <andres@anarazel.de> writes:
> Imo the code now is a bit odd, because we first switch (type) setting base,
> and then separately have branches for the different bases.

It'd be hard to merge, I think, given that the cases in the switch
don't line up one-for-one with the different bases.  You could
probably do something involving falling through between different
cases, but I think that that would be a lot harder to read;
and I'm still of the opinion that micro-optimizing this code
is probably a waste of effort for our usage.

            regards, tom lane



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

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint
Следующее
От: Robert Haas
Дата:
Сообщение: Re: Correct error message for end-of-recovery record TLI