> -----Original Message----- > From: hy...@hyrumwright.org [mailto:hy...@hyrumwright.org] On Behalf Of > Hyrum Wright > Sent: woensdag 22 september 2010 22:07 > To: Daniel Shahaf > Cc: Julian Foad; dev@subversion.apache.org > Subject: Re: svn commit: r998296 - > /subversion/trunk/subversion/include/svn_client.h > > On Wed, Sep 22, 2010 at 8:40 PM, Daniel Shahaf <d...@daniel.shahaf.name> > wrote: > > Hyrum Wright wrote on Wed, Sep 22, 2010 at 12:08:41 +0100: > >> On Wed, Sep 22, 2010 at 11:33 AM, Daniel Shahaf > <d...@daniel.shahaf.name> wrote: > >> > [1] For the [out] parameter, can we have @param[out,optional] and > >> > @param[out,mandatory] notations, or do we have to say "may be > NULL" > >> > in the prose?) > >> > >> @param[out] is part of the doxygen markup (not just some arbitrary > >> notation). I don't know what it would do in the face of extra > values > >> (see http://www.stack.nl/~dimitri/doxygen/commands.html#cmdparam) > >> > > > > Well, the [mandatory] and [optional] could be a nice extension to > that > > syntax. > > Agreed, though I'm not sure the doxygen parser would handle it. > > >> > [2] How about introducing: > >> > > >> > struct svn_ra_node_t { > >> > const char *repos_relpath; > >> > svn_revnum_t peg; > >> > };
The ra layer works in most cases session relative, where the session root is not necessary the repository root (See issue #3242) Bert