[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r40309 - trunk/subversion/svndumpfilter

From: Branko Cibej <brane_at_xbc.nu>
Date: Sat, 31 Oct 2009 14:30:00 +0100

Greg Stein wrote:
> On Fri, Oct 30, 2009 at 00:48, Branko Cibej <brane_at_xbc.nu> wrote:
>
>> ...
>> +++ trunk/subversion/svndumpfilter/main.c Thu Oct 29 21:48:12 2009 (r40309)
>> ...
>> @@ -1400,7 +1400,7 @@ main(int argc, const char *argv[])
>> style, and absolute. */
>> SVN_INT_ERR(svn_utf_cstring_to_utf8(&prefix, os->argv[i], pool));
>> prefix = svn_dirent_internal_style(prefix, pool);
>> - prefix = svn_path_join("/", prefix, pool);
>> + prefix = svn_dirent_join("/", prefix, pool);
>> APR_ARRAY_PUSH(opt_state.prefixes, const char *) = prefix;
>>
>
> Hmm? This doesn't seem to be a way to make a dirent path "absolute".
> And even then, I'd question why/what is the intent here. Are these
> prefixes actually directory entries?
>

Fixed in r40325. And thanks for the poke. I was initially just trying to
get rid of the deprecation warnings, now I actually took the time to
read svn_dirent_uri.h. :p

-- Brane

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2413235
Received on 2009-10-31 14:30:19 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.