Greg Stein <gst...@gmail.com> writes: > On Thu, Mar 3, 2011 at 10:35, Hyrum K Wright <hy...@hyrumwright.org> wrote: >> On Thu, Mar 3, 2011 at 9:03 AM, <phi...@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
