On Fri, Apr 6, 2012 at 12:44 PM, sebb <[email protected]> wrote: > On 5 April 2012 16:38, Jörg Schaible <[email protected]> wrote: > > Hi Gary, > > > > Gary Gregory wrote: > > > >> Hi All: > >> > >> I'd like to propose the change below. This lets you get rid of type > casts > >> in call sites. You'll still get a ClassCastException if you code it > wrong > >> of course. > > > > +1 > > > >> The compiler warning you'll get is valid, so do not think we should > >> @SuppressWarning against it. > > > > Suppress it. It is only for us devs and it simply means that we know the > > risk and we accept it. > > If we do add @SuppressWarning, the reason for doing so must be > documented in the code. > And the method Javadoc should warn about any Exception that might be > generated. >
Please see trunk for some fresh comments. Thank you, Gary > > > - Jörg > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > -- E-Mail: [email protected] | [email protected] JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0 Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory
