apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@gmail.com>
Subject Re: [PATCH] apr_hash_this_{key,klen,val}
Date Fri, 12 Mar 2010 11:21:47 GMT
On Tue, Mar 9, 2010 at 16:42, William A. Rowe Jr. <wrowe@rowe-clan.net> wrote:
> On 3/9/2010 3:37 PM, Hyrum K. Wright wrote:
>>
>> I'm also planning a followup which const-ifies the apr_hash_index_t params to these
functions, as well as apr_hash_this().  Thoughts?
>
> Note const'ness will only be alterable with apr 2.0 forwards.

It is *totally* fine to add a 'const' to a parameter that was not
there before. That does not change the ABI whatsoever, and it will not
break the API for callers. It merely gives them more information at
compile time.

Cheers,
-g

Mime
View raw message