Hey Martin, -----Original Message-----
From: Martin Desruisseaux <[email protected]> Organization: Geomatys Reply-To: "[email protected]" <[email protected]> Date: Tuesday, April 2, 2013 9:26 AM To: "[email protected]" <[email protected]> Subject: Re: FallbackConverterTest bug? >Hello Chris > >Le 02/04/13 17:16, Mattmann, Chris A (388J) a écrit : >> Sure, lines 71, 81, 92, 104 and 118. >Thanks. It seems to be the same issue repeated for all those lines... > >>> does this error occurs only in Eclipse, or does it occurs also in a >>>"mvn install" build? >> (...snip...) >> >> Build success with Maven3. Odd.. > >No so odd actually. Eclipse does not use the Sun/Oracle javac compiler. >It uses its own compiler instead, which doesn't behave exactly the same >way. I have seen 3 or 4 similar situations in the past, where the >Eclipse and Sun compilers were in disagreement. However I do not >remember having seen a case where the Eclipse compiler was right... Heh. > >In this particular case, the Eclipse compiler complains about that line: > >c = FallbackConverter.merge(c, new StringConverter.Long()); > >The 'merge' method expect two arguments of type: > > ObjectConverter<S, ? extends T> > ObjectConverter<S, ? extends T> > >The types that we are actually providing are: > > ObjectConverter<String, ?> > ObjectConverter<String, Long> > >If we understand <?> as synonymous to <? extends Object>, the above is >right as far as I can see. But maybe Eclipse wants it to be specified >explicitly. could you try to edit line 63 please, replacing the ><String,?> by <String,? extends Object> and see if it fixes the errors? Sure does fix them. Should I file an issue and commit? Cheers, Chris > > Thanks, > > Martin > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Chris Mattmann, Ph.D. Senior Computer Scientist NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA Office: 171-266B, Mailstop: 171-246 Email: [email protected] WWW: http://sunset.usc.edu/~mattmann/ ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Adjunct Assistant Professor, Computer Science Department University of Southern California, Los Angeles, CA 90089 USA ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
