Re: implicit casts from void*

From: John Naylor <johncnaylorls(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: implicit casts from void*
Date: 2025-07-01 05:56:57
Message-ID: CANWCAZa-jv-c7p2e0LeCSUAZ3S0tfUiGExJCwpoEGFVLAk6kKA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jul 1, 2025 at 10:36 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> John Naylor <johncnaylorls(at)gmail(dot)com> writes:
> > I received on off-list report that commit e2809e3a101 causes an error
> > when building an extension written in C++, since $subject is in a
> > header file. The fix is simply to add an explicit cast, so I plan to
> > push the attached soon.
>
> Hmpfh. No objection to your patch, but I wonder why
> "headerscheck --cplusplus" didn't find this? Can we get it
> to do so?

Good question, and it turns out it catches it just fine, but you have
to configure with CPPFLAGS="-msse4.2" (or run the script on a Red Hat
9-ish system).

--
John Naylor
Amazon Web Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Pyhalov 2025-07-01 06:43:04 Re: CREATE INDEX CONCURRENTLY on partitioned index
Previous Message Dilip Kumar 2025-07-01 05:23:40 Re: Conflict detection for update_deleted in logical replication