Greg Stein wrote:
> On Thu, Oct 9, 2008 at 9:35 AM, <lgo_at_tigris.org> wrote:
>> ...
>> +++ trunk/subversion/include/svn_path.h Thu Oct 9 09:35:13 2008 (r33582)
>> @@ -234,12 +234,8 @@ svn_dirent_is_root(const char *dirent, a
>> const char *
>> svn_path_canonicalize(const char *path, apr_pool_t *pool);
>>
>> -/** Return @c TRUE iff path is canonical. Use @a pool for temporary
>> - * allocations.
>> - *
>> - * @note The test for canonicalization is currently defined as
>> - * "looks exactly the same as @c svn_path_canonicalize() would make
>> - * it look".
>> +/** Return @c TRUE iff path is canonical. The @a pool is not used and
>> + * may be NULL.
>
> The pool is used in 1.5, right? So if somebody codes to work with 1.5
> and passes pool==NULL, then it'll crash.
This function is to be deprecated in the near future, but you're right
nonetheless.
Lieven
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-10-13 17:36:28 CEST