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 190223.1635298794@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint  (Japin Li <japinli@hotmail.com>)
Ответы Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint  (Arjan van de Ven <arjan@linux.intel.com>)
Список pgsql-hackers
Дерево обсуждения 
Japin Li <japinli@hotmail.com> writes:
> Why do we need likely() for base=10, however, base=16 and base=8 don't need?

Yeah, I was a little unconvinced about that too.  I concur with writing
it as an if/else chain instead of a switch, but I'm not sure that likely()
adds anything to that.

            regards, tom lane

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

Предыдущее
От: Japin Li
Дата:
Сообщение: Re: [PATCH v2] src/port/snprintf.c: Optimize the common base=10 case in fmtint
Следующее
От: Japin Li
Дата:
Сообщение: Re: [Bug] Logical Replication failing if the DateStyle is different in Publisher & Subscriber