Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (nike.apache.org: domain of philip.martin@wandisco.com
 designates 74.125.82.51 as permitted sender)
From: Philip Martin <philip.martin@wandisco.com>
To: Branko =?utf-8?Q?=C4=8Cibej?= <brane@apache.org>
Cc: dev@subversion.apache.org
Subject: Re: Separating deprecated code.
References: <-3477464173635182231@unknownmsgid> <51AC63B3.9090607@apache.org>
Date: Mon, 03 Jun 2013 12:09:18 +0100
In-Reply-To: <51AC63B3.9090607@apache.org> ("Branko =?utf-8?Q?=C4=8Cibej?=
 =?utf-8?Q?=22's?= message of "Mon,
	03 Jun 2013 11:36:51 +0200")
Message-ID: <8738szh2n5.fsf@ntlworld.com>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Branko =C4=8Cibej <brane@apache.org> writes:

> On 03.06.2013 11:30, Bert Huijben wrote:
>> This function is also used from deprecated code in libsvn_cli=C3=ABnt and
>> some binding code, unlike most other deprecated code...
>>
>> In some ways this code is 'a bit less deprecated' ;-)
>
> And anyway, only public APIs can be deprecated. Private APIs are either
> kept or removed as necessary.

We don't currently have any private deprecated APIs but that's just
because we haven't yet needed any.  If I move svn_wc__get_tree_conflict
and svn_wc__status2_from_3 to libsvn_wc/deprecated.c the only problem
with adding deprecated markers is where the testsuite calls one of the
functions:

Index: ../src/subversion/include/private/svn_wc_private.h
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- ../src/subversion/include/private/svn_wc_private.h  (revision 1488952)
+++ ../src/subversion/include/private/svn_wc_private.h  (working copy)
@@ -292,6 +292,7 @@
  * used to access @a victim_path.  Allocate @a *tree_conflict in @a result=
_pool,
  * use @a scratch_pool for temporary allocations.
  */
+SVN_DEPRECATED
 svn_error_t *
 svn_wc__get_tree_conflict(const svn_wc_conflict_description2_t **tree_conf=
lict,
                           svn_wc_context_t *wc_ctx,
@@ -371,6 +372,7 @@
  * Convert from svn_wc_status3_t to svn_wc_status2_t.
  * Allocate the result in RESULT_POOL.
  */
+SVN_DEPRECATED
 svn_error_t *
 svn_wc__status2_from_3(svn_wc_status2_t **status,
                        const svn_wc_status3_t *old_status,


Index: ../src/subversion/tests/libsvn_wc/conflict-data-test.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- ../src/subversion/tests/libsvn_wc/conflict-data-test.c      (revision 1=
488952)
+++ ../src/subversion/tests/libsvn_wc/conflict-data-test.c      (working co=
py)
@@ -23,6 +23,8 @@
  * conflict-data-test.c -- test the storage of tree conflict data
  */
=20
+#define SVN_DEPRECATED
+
 #include <stdio.h>
 #include <string.h>
 #include <apr_hash.h>
@@ -339,6 +341,7 @@
   {
     const svn_wc_conflict_description2_t *read_conflict;
=20
+    /* SVN_DEPRECATED API. */
     SVN_ERR(svn_wc__get_tree_conflict(&read_conflict, sbox.wc_ctx,
                                       child1_abspath, pool, pool));
     SVN_ERR(compare_conflict(read_conflict, conflict1));

--=20
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download