Re: svn commit: r1076645 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

Greg Stein <gstein_at_gmail.com> writes:

> On Thu, Mar 3, 2011 at 10:35, Hyrum K Wright <hyrum_at_hyrumwright.org> wrote:
>> On Thu, Mar 3, 2011 at 9:03 AM,  <philip_at_apache.org> wrote:
>>> Author: philip
>>> Date: Thu Mar  3 15:03:42 2011
>>> New Revision: 1076645
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1076645&view=rev
>>> Log:
>>> Wrap pointer in a baton to avoid a complier warning or cast.
>>
>> I think a single cast is better than the obfuscation of wrapping a
>> single value in a baton.  Please reconsider this change.
>
> Agreed. I'd much rather see (void *)changelist, than all of this stuff.

A cast that changes the type, (void *)changelist, or one that just
removes the qualifier, (char *)changelist? Casts lead to questions.

-- 
Philip