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

Re: svn commit: r1125327 - /subversion/trunk/subversion/svndumpfilter/main.c

From: Hyrum K Wright <hyrum_at_hyrumwright.org>
Date: Fri, 20 May 2011 15:36:29 -0700

This may be picking nits, but is the format for size_t and apr_size_t
guaranteed to be the same on all platforms?

-Hyrum

On Fri, May 20, 2011 at 4:17 AM, <philip_at_apache.org> wrote:
> Author: philip
> Date: Fri May 20 11:17:58 2011
> New Revision: 1125327
>
> URL: http://svn.apache.org/viewvc?rev=1125327&view=rev
> Log:
> * subversion/svndumpfilter/main.c
>  (write_prop_to_stringbuf): Followup to r1125288, make format match type.
>
> Modified:
>    subversion/trunk/subversion/svndumpfilter/main.c
>
> Modified: subversion/trunk/subversion/svndumpfilter/main.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svndumpfilter/main.c?rev=1125327&r1=1125326&r2=1125327&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/svndumpfilter/main.c (original)
> +++ subversion/trunk/subversion/svndumpfilter/main.c Fri May 20 11:17:58 2011
> @@ -90,7 +90,7 @@ write_prop_to_stringbuf(svn_stringbuf_t
>   namelen = strlen(name);
>   svn_stringbuf_appendbytes(*strbuf, "K ", 2);
>
> -  bytes_used = apr_snprintf(buf, sizeof(buf), "%d", namelen);
> +  bytes_used = apr_snprintf(buf, sizeof(buf), "%" APR_SIZE_T_FMT, namelen);
>   svn_stringbuf_appendbytes(*strbuf, buf, bytes_used);
>   svn_stringbuf_appendbyte(*strbuf, '\n');
>
>
>
>
Received on 2011-05-21 00:36:59 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.