"Rui, Guo" <timmyguo_at_mail.ustc.edu.cn> writes:
>> This isn't your code, but I see a lot of "if (*eb->target)" in this
>> file, which makes me think that when there is no target, eb->target is
>> "" rather than NULL. Which is fine, but we should document that in
>> struct edit_baton!
>
> It should have been ensured by svn_wc_adm_open_anchor(), I think. The document
> about anchor in notes/ directory has some words on it, if I recall correctly.
I've documented it now, see r31916.
>> The 'current_entry' here is just the parent's entry for the subdir,
>> right? And therefore the only reason we're using svn_depth_infinity is
>> so that that subdir entry won't write out an explicit depth, and the
>> subdir itself will know its own actual depth? (Just checking my own
>> understanding.)
>
> Yes. The depth field was only stored in this_dir entry. And I extend it
> that the subdir entry may have an 'exclude' depth value indicating that it is
> cropped out. However, the old depth values are still handled in the usual
> way.(in other words, restrictive values other 'exclude' is not allowed in
> subdir entries. May I explicitly check this in read_entry()?)
Yes, in fact I think that would be a good idea! :-)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-30 03:38:15 CEST