From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Add pg_strnlen() a portable implementation of strlen. |
Date: | 2017-10-10 21:56:18 |
Message-ID: | 20171010215618.qbracvhslzvuxilv@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 2017-10-10 13:53:15 -0700, Andres Freund wrote:
> On 2017-10-10 16:51:39 -0400, Tom Lane wrote:
> > Andres Freund <andres(at)anarazel(dot)de> writes:
> > > As far as I can tell it's still somehow using a configure from before
> > > the last commits:
> >
> > No, it's pilot error. The AC_CHECK_FUNCS call you added strnlen to
> > is only executed if
> > AS_IF([test "$enable_thread_safety" = yes -a "$PORTNAME" != "win32"],
>
> Dear god, I'm daft. Dear god, m4 is a horrible lanuagage.
Here's a fix. Not quite sure whether we really need the
HAVE_DECL_STRNLEN test, added it for symmetry.
Afaict windows "always" had strnlen, so no need to meddle with the
windows build.
Will eat lunch and push, even if there's some futher adjustments, I'd
like to get Andrew's animals green again.
Greetings,
Andres Freund
Attachment | Content-Type | Size |
---|---|---|
0001-Rewrite-strnlen-replacement-implementation-from-8a24.patch | text/x-diff | 9.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-10-10 21:59:50 | Re: pgsql: Add pg_strnlen() a portable implementation of strlen. |
Previous Message | Andrew Dunstan | 2017-10-10 21:34:17 | Re: pgsql: Add pg_strnlen() a portable implementation of strlen. |