Arfrever Frehtes Taifersar Arahesis wrote:
> [[[
> Mark deprecated functions with SVN_DEPRECATED.
>
> * subversion/include/svn_types.h: Define SVN_DEPRECATED.
>
> * subversion/bindings/swig/include/svn_global.swg: Define __declspec(x).
Can you comment on why this is necessary? It doesn't look like the
"__declspec" would be seen by SWIG, given that in "svn_types.h" you have
"#if !defined(SWIG..." which tries to hide it from SWIG.
> * subversion/include/svn_auth.h:
> * subversion/include/svn_base64.h:
> * subversion/include/svn_client.h:
> * subversion/include/svn_cmdline.h:
> * subversion/include/svn_config.h:
> * subversion/include/svn_delta.h:
> * subversion/include/svn_diff.h:
> * subversion/include/svn_dso.h:
> * subversion/include/svn_error.h:
> * subversion/include/svn_fs.h:
> * subversion/include/svn_hash.h:
> * subversion/include/svn_io.h:
> * subversion/include/svn_opt.h:
> * subversion/include/svn_ra.h:
> * subversion/include/svn_ra_svn.h:
> * subversion/include/svn_repos.h:
> * subversion/include/svn_subst.h:
> * subversion/include/svn_utf.h:
> * subversion/include/svn_wc.h: Mark deprecated functions with SVN_DEPRECATED.
It's probably not such a good idea to combine adding "SVN_DEPRECATED"
with moving the return type to a separate line in the same change. That
will make it harder to merge changes between branches. Can you separate
these two changes please?
> * www/hacking.html
> (deprecation): Update example.
> ]]]
Other than that, it looks good. I tried it (with GCC 4.1.2) and confirm
it does what it should.
Unfortunately, GCC 4.1.2 emits a warning for using a deprecated function
even where this use is in the body of another deprecated function, which
is not what I want, but that's another matter.
- Julian
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-26 11:37:06 CEST