[modules-dev] Review request for 6588155 (fixes to borken tests)

Dave Bristor David.Bristor at Sun.COM
Thu Aug 2 14:31:34 PDT 2007


Andreas Sterbenz wrote:
> Dave Bristor wrote:
>>
>> * RepositoryConfig: Now yields useful output from the 
>> InvocationTargetException.
> 
> I am not sure about putting the entire stack trace into the exception 
> message. That's not what it was designed for. What's wrong with chaining 
> the cause, i.e. new IllegalArgumentException(Cannot create ...", 
> ex.getCause()) ?

I've just done the putback: FWIW, I left in the printing of the entire stack 
trace; I can remove it but (a) wanted to get this putback done and (b) found 
the stack useful in tracking down the problem.

	Dave

> 
> Rest looks fine.
> 
> Andreas.
> 



More information about the modules-dev mailing list