Re: unconstify equivalent for volatile - Mailing list pgsql-hackers

From Petr Jelinek
Subject Re: unconstify equivalent for volatile
Date
Msg-id 03081802-8268-cc86-fab7-e5562d54532d@2ndquadrant.com
Whole thread Raw
In response to Re: unconstify equivalent for volatile  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: unconstify equivalent for volatile  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 18/02/2019 16:43, Tom Lane wrote:
> Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
>> I propose to add an equivalent to unconstify() for volatile.  When
>> working on this, I picked the name unvolatize() mostly as a joke, but it
>> appears it's a real word.  Other ideas?
> 
> Umm ... wouldn't this amount to papering over actual bugs?  I can
> think of legitimate reasons to cast away const, but casting away
> volatile seems pretty dangerous, and not something to encourage
> by making it notationally easy.
> 

I'd argue that it's not making it easier to do but rather easier to spot
(vs normal type casting) which is IMO a good thing from patch review
perspective.

-- 
  Petr Jelinek                  http://www.2ndQuadrant.com/
  PostgreSQL Development, 24x7 Support, Training & Services


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Missing Column names with multi-insert
Next
From: Tom Lane
Date:
Subject: Re: unconstify equivalent for volatile