From: | Ranier Vilela <ranier(dot)vf(at)gmail(dot)com> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Replace some cstring_to_text to cstring_to_text_with_len |
Date: | 2023-08-31 14:10:55 |
Message-ID: | CAEudQAoyoRb=zyVWRFa0t3qNzghTaiM7Wc-q-U6OMjZL7Ks8bA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Em qui., 31 de ago. de 2023 às 09:51, Andrew Dunstan <andrew(at)dunslane(dot)net>
escreveu:
>
> On 2023-08-31 Th 07:41, John Naylor wrote:
>
>
> On Thu, Aug 31, 2023 at 6:07 PM Ranier Vilela <ranier(dot)vf(at)gmail(dot)com> wrote:
> >
> > Em qui., 31 de ago. de 2023 às 00:22, Michael Paquier <
> michael(at)paquier(dot)xyz> escreveu:
> >>
> >> On Wed, Aug 30, 2023 at 03:00:13PM -0300, Ranier Vilela wrote:
> >> > cstring_to_text has a small overhead, because call strlen for
> >> > pointer to char parameter.
> >> >
> >> > Is it worth the effort to avoid this, where do we know the size of the
> >> > parameter?
> >>
> >> Are there workloads where this matters?
> >
> > None, but note this change has the same spirit of 8b26769bc.
>
> - return cstring_to_text("");
> + return cstring_to_text_with_len("", 0);
>
> This looks worse, so we'd better be getting something in return.
>
>
> I agree this is a bit ugly. I wonder if we'd be better off creating a
> function that returned an empty text value, so we'd just avoid converting
> the empty cstring altogether and say:
>
> return empty_text();
>
Hi,
Thanks for the suggestion, I agreed.
New patch is attached.
best regards,
Ranier Vilela
>
Attachment | Content-Type | Size |
---|---|---|
0001-Replace-cstring_to_text-with-empty_text.patch | application/octet-stream | 7.0 KB |
0002-Avoid-unecessary-calls-to-strlen-function.patch | application/octet-stream | 3.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ranier Vilela | 2023-08-31 14:11:44 | Re: Replace some cstring_to_text to cstring_to_text_with_len |
Previous Message | Ranier Vilela | 2023-08-31 14:09:48 | Re: Replace some cstring_to_text to cstring_to_text_with_len |