On Mon, 5 Jun 2023 21:43:20 GMT, Jonathan Gibbons <j...@openjdk.org> wrote:

>> Pavel Rappo has updated the pull request with a new target base due to a 
>> merge or a rebase. The incremental webrev excludes the unrelated changes 
>> brought in by the merge/rebase. The pull request contains three additional 
>> commits since the last revision:
>> 
>>  - Merge branch 'master' into 8304878
>>  - Check exceptions explicitly
>>  - Initial commit
>
> test/langtools/jdk/javadoc/doclet/testLatePackageDiscovery/TestLatePackageDiscovery.java
>  line 65:
> 
>> 63:                 "--module", "test.foo",
>> 64:                 "-nodeprecated");
>> 65:         checkExit(Exit.OK);
> 
> (Optional)
> Since this is about ensuring we don't get CME again,
> either explicitly turn on exception checking (even though it is on by default)
> or insert a comment noting than exception checking is on by default

Done in db1509b389f.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14310#discussion_r1219176939

Reply via email to