Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> On 10/31/17 10:56, Tom Lane wrote:
>>> Some functions have a strtol()-like behavior
>>> where they take in a const char * and return a pointer into that as
>>> another argument. In those cases, I added a cast or two.
>> ... but I'm not sure that it's an improvement in cases where you have to
>> cast away the const somewhere else. I realize that strtol has an ancient
>> pedigree, but I do not think it's very good design.
> Would you prefer leaving the input argument as char *, or change the
> endptr argument to const as well?
Just leave it as char*. If you change the endptr argument you're going to
force every call site to change their return variable, and some of them
would end up having to cast away the const on their end.
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers