Hi Bertrand,

On Fri, Nov 24, 2023 at 6:26 AM Drouvot, Bertrand <
bertranddrouvot...@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.

Attachment: v2-0001-Don-t-copy-bytea-text-in-convert_from-to-unless-conv.patch
Description: Binary data

Reply via email to