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

Re: Diff Project --invoke-diff-cmd part

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Fri, 12 Apr 2013 01:11:20 +0100 (BST)

Daniel Shahaf wrote:

>> + * @deprecated Provided for backward compatibility with the 1.8 API.
>>   * @since New in 1.8.
>>   */
>> +SVN_DEPRECATED
>>   svn_error_t *
>>   svn_client_diff6(const apr_array_header_t *diff_options,
>
> Since this API is new in 1.8, you should just extend it without
> deprecating it.
>
> (We only deprecate released APIs.  svn_client_diff5 is released and set
> in stone (save for certain const-ness and pointed-to structs changes,
> maybe); svn_client_diff6 has not been released and can be modified
> arbitrarily.  (but those changes should be added to the "SVN 1.8 APIs
> review" wiki page))

In principle, what Daniel states is correct, but in practice we're intending to branch 1.8 tomorrow, so this work is destined for 1.9, so leave that as it is (and change your new functions to say "New in 1.9.").

- Julian
Received on 2013-04-12 02:11:53 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.