Re: svn commit: r34214 - in trunk/subversion: include libsvn_subr
On Sat, Nov 15, 2008 at 2:17 AM, Branko ╚ibej <brane_at_xbc.nu> wrote:
> Greg Stein wrote:
>>>> We don't want a svn_cstring_join2(), do we?
>>>> Instead, we could cast explicitly to remove the warnings from gstein's r34178.
>> No need. This is a valid change. From a developer's standpoint, they
>> can now pass either non-const (as before) or const arrays into
>> svn_cstring_join(). We're *adding* capability, but not taking away (or
>> making source incompatible) anything they had before. At a binary
>> level, nothing has changed either. We're simply telling developers
>> "hey, we won't change what you pass to us."
>> Totally safe change.
> So I'd have thought. But I recall at least once when this came up,
> someone argued that adding const is potentially an ABI change on some
> architectures that may pass const arguments differently.
> So which is it?
Wow. Never heard of that. But hey... I guess that is possible.
Not so sure now...
Received on 2008-11-15 12:35:26 CET
This is an archived mail posted to the Subversion Dev