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

Separating deprecated code.

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Mon, 03 Jun 2013 09:32:51 +0100

Philip Martin <philip.martin_at_wandisco.com> writes:

> Barry Scott <barry_at_barrys-emacs.org> writes:
>
>> Index: /Users/barry/wc/svn/svn-1.8.x/subversion/libsvn_wc/tree_conflicts.c
>> ===================================================================
>> --- /Users/barry/wc/svn/svn-1.8.x/subversion/libsvn_wc/tree_conflicts.c (revision 1484244)
>> +++ /Users/barry/wc/svn/svn-1.8.x/subversion/libsvn_wc/tree_conflicts.c (working copy)
>> @@ -506,6 +506,7 @@
>> return SVN_NO_ERROR;
>> }
>> }
>> + *tree_conflict = NULL;
>> return SVN_NO_ERROR;
>> }
>
> Committed to trunk and nominated for backport. Thanks!

This is a patch to svn_wc__get_tree_conflict which is only called from
util.c:svn_wc__status2_from_3 which is itself only called in the main
code from libsvn_wc/deprecated.c and libsvn_client/deprecated.c. That
makes both svn_wc__get_tree_conflict and svn_wc__status2_from_3 only
necessary for support of the deprecated API.

Should we move the code to deprecated.c? Should we add DEPRECATED
markings? conflict-data-test.c also calls svn_wc__get_tree_conflict
which makes it an explicit test of the deprecated code. If we mark the
functions DEPRECATED do we explicitly disable DEPRECATED warnings in the
test file?

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2013-06-03 10:33:27 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.