On Thu, 14 Mar 2024 17:35:22 GMT, Mandy Chung <mch...@openjdk.org> wrote:

> Trivial fix.  Improve the error message to print the cause of the module 
> resolution failure if present.

Hello Mandy, this change to use the underlying cause of `ResolutionExcepion` 
and `FindException` when reporting the error message, appears OK to me.

I didn't know if `ResolutionException` too wraps the actual cause underneath, 
so I looked around a few JBS issues where `ResolutionException` was reported as 
being thrown and it does appear that the actual cause lies underneath. So I 
think this change is fine.

-------------

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18308#pullrequestreview-1937969339

Reply via email to