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

Re: 1.8 new public API review (mostly) complete.

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Mon, 1 Apr 2013 11:25:49 -0400

On 03/31/2013 12:31 AM, Blair Zajac wrote:
> On 03/29/2013 01:17 PM, C. Michael Pilato wrote:
>> Devs,
>>
>> I've just completed my review of the new-in-1.8 public APIs, minus the bits
>> that Philip reviewed (thanks!) and the new merge-related stuff which, if I
>> understand from recent threads correctly, is still subject to some churn.
>>
>> The results of my review revealed overwhelmingly positive results which, in
>> my approximation, are non-contentious. I had (for some definition of "had")
>> to touch up quite a few docstring in the process, but by and large those
>> were stylistic nits with the occasionally overlooked item.
>
> Question on svn_repos_load_fs4() and svn_repos_get_fs_build_parser4() (maybe
> other functions have a similar text):
>
> * @note If @a start_rev and @a end_rev are valid revisions, this
> * function presumes the revisions as numbered in @a dumpstream only
> * increase from the beginning of the stream to the end. Gaps in the
> * number sequence are ignored, but upon finding a revision number
> * younger than the specified range, this function may stop loading
> * new revisions regardless of their number.
>
> What does 'may stop' mean? Does it flips a coin ;) Seriously, will it or
> will it not stop, or under which conditions.

Heh. It's been a long while -- I'll look into that again. On first (again)
glance, I don't see any code which would stop anything.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development

Received on 2013-04-01 17:26:25 CEST

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