Re: Large writable variables - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Large writable variables
Date
Msg-id 200cf391-4a60-4c69-85d8-c715c7bce4de@2ndquadrant.com
Whole thread Raw
In response to Re: Large writable variables  (Andres Freund <andres@anarazel.de>)
Responses Re: Large writable variables
List pgsql-hackers
On 16/10/2018 08:30, Andres Freund wrote:
> This just reminded me that a couple times I wanted a cast that casts
> away const, but otherwise makes sure the type stays the same. I don't
> think there's a way to do that in C, but we can write one that verifies
> the cast doesn't do something bad if gcc is used:
> 
> #if defined(HAVE__BUILTIN_TYPES_COMPATIBLE_P)
> #define unconstify(cst, var) StaticAssertExpr(__builtin_types_compatible_p(__typeof(var), const cst), "wrong cast"),
(cst)(var)
 
> #else
> #define unconstify(cst, var) ((cst) (var))
> #endif
> 
> Does anybody besides me see value in adding a cleaned up version of
> that?

I've had the attached patch lying around.  As you can see there, there
are a few places where it could be useful, but not too many.

The name CONST_CAST() is adapted from C++.

Your version with __builtin_types_compatible_p() doesn't work for
casting between char * and const char *, so it wouldn't be very useful.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment

pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: pg_dump multi VALUES INSERT
Next
From: Tom Lane
Date:
Subject: Re: Large writable variables