On Sat, 23 Jul 2022 19:41:07 GMT, Leonid Mesnik <[email protected]> wrote:
> Fixed tests that use eName to print event name to print the class name. Plus
> some minor refactoring.
Changes requested by cjplummer (Reviewer).
test/hotspot/jtreg/vmTestbase/nsk/jdi/Event/request/request001.java line 560:
> 558: if ((flagsCopy & flag) == 0) {
> 559: log3("ERROR: event duplication. flafsCopy = " +
> Integer.toBinaryString(flagsCopy)
> 560: + " flag = " + Integer.toBinaryString(flag));
typo: flafsCopy
I think you should print the event also.
test/hotspot/jtreg/vmTestbase/nsk/jdi/LocatableEvent/thread/thread001.java line
448:
> 446: throw new JDITestRuntimeException("** unexpected
> event **");
> 447: }
> 448: log2("--------> got: " + event1.getClass().getName() + "
> index: " + index);
Why not print the event rather than just its name.
-------------
PR: https://git.openjdk.org/jdk/pull/9624