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

RE: svn commit: r35512 - in trunk/subversion: include libsvn_client svn

From: Bert Huijben <rhuijben_at_sharpsvn.net>
Date: Thu, 29 Jan 2009 16:14:37 +0100

> -----Original Message-----
> From: Bert Huijben [mailto:rhuijben_at_sharpsvn.net]
> Sent: woensdag 28 januari 2009 17:15
> To: 'C. Michael Pilato'; 'Bert Huijben'
> Cc: 'Greg Stein'; dev_at_subversion.tigris.org
> Subject: RE: svn commit: r35512 - in trunk/subversion: include
> libsvn_client svn

> Which others would you like to revert too?
> Feel free to revert..
>
> svn_client_status4: r35518,r35515-r35514,r35512
> svn_client_log5: r35242,r35245

I added a non blocking note to TODO-1.6 with the 4 options of which we
should choose one before branching 1.6:
* revert the svn_client_status set
* revert the svn_client_status and svn_client_log set under the assumption
that we just create new functions later for both cases.

And the last options:
* switch to bitflags for the booleans. (I don't think we use bitflags
anywhere, but these have the same documentation advantage).
* move more arguments to svn_client_status_args_t to make the struct more
usefull than just keeping 3 booleans.

I would like some response in this thread from others before just choosing
the first (status only revert) option myself. Julian and Hyrum were +1 on
the initial ideas and Hyrum introduced svn_client_log_args_t.

        Bert

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1069120
Received on 2009-01-29 16:15:23 CET

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