[weld-issues] [JBoss JIRA] Resolved: (WELDX-76) Use the valditor's message in HelloWorld in SE Hello World Example

[Peter Royle (JIRA)]

     [ https://jira.jboss.org/jira/browse/WELDX-76?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Peter Royle resolved WELDX-76.
------------------------------

    Fix Version/s: Java SE 1.0.1
       Resolution: Done


Fixed

> Use the valditor's message in HelloWorld in SE Hello World Example
> ------------------------------------------------------------------
>
>                 Key: WELDX-76
>                 URL: https://jira.jboss.org/jira/browse/WELDX-76
>             Project: Weld Extensions
>          Issue Type: Feature Request
>          Components: Java SE
>         Environment: n/a
>            Reporter: Wolfgang Frech
>            Assignee: Peter Royle
>             Fix For: Java SE 1.0.1
>
>
> It looks like an incomplete refactoring: 
> The CommandLineArgsValidator creates an error message, but the HellWorld class does not read the message from the validator, but prints its own (identical) message.
> Use
>            for (String message: argsValidator.getErrors())
>             System.out.println( message );
> instead of
> System.out.println( "Please provide just one argument: your first name" );
> in printHello
> by the way - cosmetic: use emptyList instead of deprecated EMPTY_LIST in CommandLineArgsValidator in SE Hello World Example

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira