Hi Alex,
On 14/05/2020 12:20 pm, Alex Menkov wrote:
Hi all,
please review tiny (and I suppose trivial) fix for
https://bugs.openjdk.java.net/browse/JDK-8244973
webrev:
http://cr.openjdk.java.net/~amenkov/jdk15/RemovingUnixDomainSocket_merge/webrev/
This bug is a result of bad merge:
base:
out.stderrShouldBeEmpty();
fix for JDK-8242009:
- out.stderrShouldBeEmpty();
+ out.stderrShouldBeEmptyIgnoreVMWarnings();
fix for JDK-8235211:
- out.stderrShouldBeEmpty();
+ out.shouldHaveExitValue(0)
+ .stderrShouldBeEmpty();
Merge fix looks good but see below ...
Nit: for this style of chained call you should align the dots (as we do
for stream operations):
out.shouldHaveExitValue(0)
.stderrShouldBeEmptyIgnoreVMWarnings();
Test run is in progress.
... I'm puzzled by the failure mode as I see:
----------System.err:(23/1191)----------
[jcmd] java version "15-ea" 2020-09-15
stdout: [Java HotSpot(TM) 64-Bit Server VM version 15-ea+23-1102
JDK 15.0.0
];
stderr: [Java(TM) SE Runtime Environment (fastdebug build 15-ea+23-1102)
Java HotSpot(TM) 64-Bit Server VM (fastdebug build 15-ea+23-1102, mixed
mode)
]
and these are not VM warnings.
Thanks,
David
--alex
