Re: svn commit: r1535399 - in /subversion/trunk/subversion: libsvn_fs_fs/ libsvn_fs_x/ libsvn_subr/ libsvn_wc/ mod_authz_svn/ tests/cmdline/ tests/libsvn_client/ tests/libsvn_fs_fs/ tests/libsvn_fs_x/ tests/libsvn_subr/ tests/libsvn_wc/

Branko Čibej <brane_at_wandisco.com> writes:

> On 25.10.2013 11:40, Philip Martin wrote:
>> Branko Čibej <brane_at_wandisco.com> writes:
>>
>>> Sideline: *IF* we, as a community, agree to do this, then: (a) the rule
>>> should be documented in HACKING, and (b) there should be a '#define
>>> NULLPTR ((void*)0)' or reasonable facsimile in either svn_types.h or
>>> svn_private_config.h. I really do not want to see '(char*)NULL' all over
>>> the place. That's ... unsightly.
>> That would be incompatible with C++ since (void*) doesn't automatically
>> convert to (char*) by default.
>
> You do realize that this is entirely irrelevant for varargs, don't you?

Ah, yes, that definition of NULLPTR would work for varargs.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*