[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