> Short version:
> Stop testing this test with -Xcomp by adding requires vm.compMode != "Xcomp"
> 
> Make additional typo fixes and tidyups while here, it's just shocking.
> 
> TestDescription.java contains the test definition, so the "requires" goes 
> there, with a comment.
> 
> Updates to from001.java are typos and clarifications, and a changed loop with 
> a poll on a queue rather than block forever.

Kevin Walls 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 four additional commits since 
the last revision:

 - Merge remote-tracking branch 'upstream/master' into 
8332551_MemoryNotificationInfo_from001
 - formatting
 - update
 - 8332551: Test 
vmTestbase/nsk/monitoring/MemoryNotificationInfo/from/from001/TestDescription.java
 timed out

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/20146/files
  - new: https://git.openjdk.org/jdk/pull/20146/files/a6965231..bcf6b450

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=20146&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=20146&range=01-02

  Stats: 8630 lines in 367 files changed: 5704 ins; 1620 del; 1306 mod
  Patch: https://git.openjdk.org/jdk/pull/20146.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/20146/head:pull/20146

PR: https://git.openjdk.org/jdk/pull/20146

Reply via email to