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

Re: svn commit: r1396432 - /subversion/trunk/subversion/svn/file-merge.c

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Wed, 23 Jan 2013 13:41:20 +0400

On Wed, Oct 10, 2012 at 4:10 AM, <hwright_at_apache.org> wrote:
> Author: hwright
> Date: Wed Oct 10 00:10:45 2012
> New Revision: 1396432
>
> URL: http://svn.apache.org/viewvc?rev=1396432&view=rev
> Log:
> * subversion/svn/file-merge.c
> (prepare_line_for_display): Eliminate integer size mismatch warning.
>
> Modified:
> subversion/trunk/subversion/svn/file-merge.c
>
> Modified: subversion/trunk/subversion/svn/file-merge.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/svn/file-merge.c?rev=1396432&r1=1396431&r2=1396432&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/svn/file-merge.c (original)
> +++ subversion/trunk/subversion/svn/file-merge.c Wed Oct 10 00:10:45 2012
> @@ -365,7 +365,7 @@ static const char *
> prepare_line_for_display(const char *line, apr_pool_t *pool)
> {
> svn_stringbuf_t *buf = svn_stringbuf_create(line, pool);
> - int width;
> + size_t width;
This change introduces three warnings on Windows platform, because
width holds int value returned from svn_utf_cstring_utf8_with()
function:
[[
..\..\..\subversion\svn\file-merge.c(398): warning C4018: '>' :
signed/unsigned mismatch
..\..\..\subversion\svn\file-merge.c(418): warning C4018: '<' :
signed/unsigned mismatch
..\..\..\subversion\svn\file-merge.c(424): warning C4389: '==' :
signed/unsigned mismatch
]]]

-- 
Ivan Zhakov
Received on 2013-01-23 10:42:18 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.