Re: svn commit: r1076645 - /subversion/trunk/subversion/libsvn_wc/wc_db.c
On Thu, Mar 3, 2011 at 12:05 PM, Philip Martin
> Greg Stein <gstein_at_gmail.com> writes:
>> It is a lot of crap just to avoid a single cast. It makes the code
>> more complicated than it needs to be.
> I don't see the cast as an improvement but I won't object if somebody
> changes it (a bit like one variable declaration per line which I think
> is crap :)
> I'll point out that this is application memory. áIf somebody were to
> inadvertently modify the code to write through the non-const pointer
> that would be a SEGV bug waiting for an application to pass read-only
> memory. áUnlikely, I know.
Reverted r1076645 and added the cast in r1078008.
Received on 2011-03-04 16:34:50 CET
This is an archived mail posted to the Subversion Dev