Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <231527620.1231913579874.JavaMail.jira@brutus>
Date: Tue, 13 Jan 2009 22:12:59 -0800 (PST)
From: "Henri Yandell (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Updated: (CLI-170) TypeHandler prints messages to stderr
In-Reply-To: <682810045.1222528664419.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/CLI-170?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Henri Yandell updated CLI-170:
------------------------------

    Fix Version/s:     (was: 1.3)
                   1.2

> TypeHandler prints messages to stderr
> -------------------------------------
>
>                 Key: CLI-170
>                 URL: https://issues.apache.org/jira/browse/CLI-170
>             Project: Commons CLI
>          Issue Type: Improvement
>          Components: Validation
>    Affects Versions: 1.1
>            Reporter: J. Lewis Muir
>            Priority: Minor
>             Fix For: 1.2
>
>         Attachments: remove-messages-to-stderr.patch
>
>   Original Estimate: 0.5h
>  Remaining Estimate: 0.5h
>
> Certain methods in {{TypeHandler}} print a message to stderr when they are unable to create the object they are to return.  I don't think this should be.  The documentation for each method clearly states that if it is unable to create the object, it returns {{null}}.  If more information should be reported to the caller, these methods should be changed to throw an exception.  I've removed the printing of these messages from {{TypeHandler}}.
> This fix also makes the unit test output clean.  Right now, certain test cases exercise the "returns {{null}}" behavior of some of the {{TypeHandler}} methods, and this causes various error messages to be written to stderr which makes the test case output look unclean and like something has gone wrong when in fact everything is working correctly.
> Attached is a patch for this fix.  The patch is against [http://svn.apache.org/repos/asf/commons/proper/cli/branches/cli-1.x].

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.