From: | Japin Li <japinli(at)hotmail(dot)com> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | ranier(dot)vf(at)gmail(dot)com, amit(dot)kapila16(at)gmail(dot)com, "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Remove redundant strlen call in ReplicationSlotValidateName |
Date: | 2021-07-16 10:42:38 |
Message-ID: | MEYP282MB1669AC6E54DF99948185A54EB6119@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, 16 Jul 2021 at 18:05, David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
> On Fri, 16 Jul 2021 at 20:35, Japin Li <japinli(at)hotmail(dot)com> wrote:
>> > When I fix a bug about ALTER SUBSCRIPTION ... SET (slot_name) [1], Ranier Vilela
>> > finds that ReplicationSlotValidateName() has redundant strlen() call, Since it's
>> > not related to that problem, so I start a new thread to discuss it.
>
> I think this is a waste of time. The first strlen() call is just
> checking for an empty string. I imagine all compilers would just
> optimise that to checking if the first char is '\0';
>
> https://godbolt.org/z/q58EGYMfM
>
Thanks for your review, this tool is amazing. The two writes on some compiler
might be difference. As Amit Kapila said, it might be helpless for reducing
overhead.
https://godbolt.org/z/j5on6Khxb
--
Regrads,
Japin Li.
ChengDu WenWu Information Technology Co.,Ltd.
From | Date | Subject | |
---|---|---|---|
Next Message | Ranier Vilela | 2021-07-16 10:55:39 | Re: Remove redundant strlen call in ReplicationSlotValidateName |
Previous Message | Ranier Vilela | 2021-07-16 10:22:39 | Re: [PATCH] Use optimized single-datum tuplesort in ExecSort |