dev@javaserverfaces.java.net

RE: [REVIEW] Fix for spec issue 172

From: Jason Lee <lee_at_iecokc.com>
Date: Tue, 15 Aug 2006 15:40:30 -0500

Just out of curiosity, why use the break, then the test? Why not do
this?

        for (int i=0; i<enumConstants.length; i++) {
            if (enumConstants[i].equals(value)) {
                return value.toString();
            }
        }
      
        // No match found, so throw an exception
        throw new ConverterException(
            MessageFactory.getMessage(context,
                ENUM_ID,
                value,
                value,
                MessageFactory.getLabel(context,
                component)));

-----
Jason Lee
Programmer/Analyst
http://www.iecokc.com
 

> -----Original Message-----
> From: Roger.Kitain_at_Sun.COM [mailto:Roger.Kitain_at_Sun.COM]
> Sent: Tuesday, August 15, 2006 2:55 PM
> To: dev_at_javaserverfaces.dev.java.net
> Subject: [REVIEW] Fix for spec issue 172
>
> https://javaserverfaces-spec-public.dev.java.net/issues/show_b
> ug.cgi?id=172
>
> Changes attached to issue.
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_javaserverfaces.dev.java.net
> For additional commands, e-mail: dev-help_at_javaserverfaces.dev.java.net
>
>