Re: [PATCH] pg_convert improvement

Поиск
Список
Период
Сортировка
От Yurii Rashkovskii
Тема Re: [PATCH] pg_convert improvement
Дата
Msg-id CA+RLCQzAbj6yFd2-PZaMJNjSxQNyZAU_Y+Z2FLKFPbUo5iCf7w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] pg_convert improvement  ("Drouvot, Bertrand" <bertranddrouvot.pg@gmail.com>)
Ответы Re: [PATCH] pg_convert improvement  ("Drouvot, Bertrand" <bertranddrouvot.pg@gmail.com>)
Список pgsql-hackers
Hi Bertrand,

On Fri, Nov 24, 2023 at 6:26 AM Drouvot, Bertrand <bertranddrouvot.pg@gmail.com> wrote:

The patch is pretty straightforward, I just have one remark:

+       /* if no actual conversion happened, return the original string */
+       /* (we are checking pointers to strings instead of encodings because
+          `pg_do_encoding_conversion` above covers more cases than just
+          encoding equality) */

I think this could be done in one single comment and follow the preferred style
for multi-line comment, see [1].

Thank you for your feedback. I've attached a revised patch. 

--
Y.

Вложения

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

Предыдущее
От: "Drouvot, Bertrand"
Дата:
Сообщение: Re: [PATCH] pg_convert improvement
Следующее
От: Heikki Linnakangas
Дата:
Сообщение: Re: Stack overflow issue