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

Re: svn_wc_diff_callbacks4_t and abspaths

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Wed, 13 Apr 2011 13:05:00 +0200 (CEST)

> On Sat, 2011-03-26, Daniel Shahaf wrote:
>> >From doc string of svn_wc_diff_callbacks4_t:
>>
>> "Note that @a path is a relative path, not just the basename of the
>> path."
>>
>>
>> >From merge_file_added() (which implements
>> svn_wc_diff_callbacks4_t.file_added):
>>
>> SVN_ERR_ASSERT(svn_dirent_is_absolute(mine_abspath));
>>
>>
>> Given that MINE_ABSPATH is the parameter called PATH in the doc string,
>> I think the contradiction is obvious...
>
> I notice that the doc string no longer documents the form of "@a path":
> that sentence you quoted is not there; but the problem still exists.
>

It's not there because I removed it due to being false.

> It would be good to rename the function parameters in the struct to
> better names, like the names used in merge_file_added():
> "mine_[abs?]path", "older_abspath", etc. instead of "path", "tmpfile1",
> etc.
>

Okay, I can go through and update the documentation/implementors to
reflect that paths passed to this vtable should be absolute.

> - Julian
>
>
>
Received on 2011-04-13 13:05:43 CEST

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.