subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Sperling <s...@elego.de>
Subject Re: svn commit: r1693472 - /subversion/trunk/subversion/libsvn_client/resolved.c
Date Thu, 30 Jul 2015 21:04:22 GMT
On Thu, Jul 30, 2015 at 10:28:48PM +0200, Branko ─îibej wrote:
> On 30.07.2015 20:50, stsp@apache.org wrote:
> > Author: stsp
> > Date: Thu Jul 30 18:50:50 2015
> > New Revision: 1693472
> >
> > URL: http://svn.apache.org/r1693472
> > Log:
> > * subversion/libsvn_client/resolved.c
> >   (resolve_conflict): Use local variable 'err' as intended.
> >
> > Modified:
> >     subversion/trunk/subversion/libsvn_client/resolved.c
> 
> 
> .../subversion/libsvn_client/resolved.c:306:35: warning: implicit conversion from enumeration
type
>       'svn_client_conflict_option_id_t' (aka 'enum svn_client_conflict_option_id_t')
to different enumeration type
>       'svn_wc_conflict_choice_t' (aka 'enum svn_wc_conflict_choice_t') [-Wenum-conversion]
>                                   option_id, /* id is backwards compatible */
>                                   ^~~~~~~~~
> 

That's on purpose and had been in the code since before this commit.
The new enum is using the same values as the old one but the compiler
can't see that.

If this warning is too upsetting I can write a function that maps
from one enum type to another. FWIW my gcc does not give a warning.

Mime
View raw message