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

Re: svn commit: r34214 - in trunk/subversion: include libsvn_subr

From: Stefan Sperling <stsp_at_elego.de>
Date: Sat, 15 Nov 2008 02:33:33 +0000

On Fri, Nov 14, 2008 at 06:19:27PM -0800, neels_at_tigris.org wrote:
> Author: neels
> Date: Fri Nov 14 18:19:27 2008
> New Revision: 34214
>
> Log:
> Follow up on r34178, getting rid of compiler warnings.
>
> * subversion/include/svn_string.h,
> * subversion/libsvn_subr/svn_string.c
> (svn_cstring_join): Make first parameter STRINGS const.

Hmmm... can already released public API be changed like that?

Stefan

>
> Modified:
> trunk/subversion/include/svn_string.h
> trunk/subversion/libsvn_subr/svn_string.c
>
> Modified: trunk/subversion/include/svn_string.h
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/include/svn_string.h?pathrev=34214&r1=34213&r2=34214
> ==============================================================================
> --- trunk/subversion/include/svn_string.h Fri Nov 14 17:46:41 2008 (r34213)
> +++ trunk/subversion/include/svn_string.h Fri Nov 14 18:19:27 2008 (r34214)
> @@ -364,7 +364,7 @@ svn_cstring_count_newlines(const char *m
> * @since New in 1.2.
> */
> char *
> -svn_cstring_join(apr_array_header_t *strings,
> +svn_cstring_join(const apr_array_header_t *strings,
> const char *separator,
> apr_pool_t *pool);
>
>
> Modified: trunk/subversion/libsvn_subr/svn_string.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_subr/svn_string.c?pathrev=34214&r1=34213&r2=34214
> ==============================================================================
> --- trunk/subversion/libsvn_subr/svn_string.c Fri Nov 14 17:46:41 2008 (r34213)
> +++ trunk/subversion/libsvn_subr/svn_string.c Fri Nov 14 18:19:27 2008 (r34214)
> @@ -569,7 +569,7 @@ int svn_cstring_count_newlines(const cha
> }
>
> char *
> -svn_cstring_join(apr_array_header_t *strings,
> +svn_cstring_join(const apr_array_header_t *strings,
> const char *separator,
> apr_pool_t *pool)
> {
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe_at_subversion.tigris.org
> For additional commands, e-mail: svn-help_at_subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-11-15 03:34:04 CET

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.