Details
-
Improvement
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
1.1
-
None
Description
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.