On 4/10/18 8:34 AM, Lindenmaier, Goetz wrote:
thanks for doing this laborious change ... comparing all these files :)
Change looks good, just some minor comments:
You say you are sorting the includes, but in the VirtualMachineImpl.c
files the order is changed, but according to which order? It's
not alphabetical as in other files.
In windows VirtualMachineImpl.c, what was wrong with printing the
last error code?
JNU_ThrowIOExceptionWithLastError already includes it.
From: serviceability-dev [mailto:serviceability-dev-
boun...@openjdk.java.net] On Behalf Of Langer, Christoph
Sent: Freitag, 6. April 2018 17:02
Subject: [CAUTION] RFR (M): 8201247: Various cleanups in the attach
can I please get reviews for a set of clean up changes that I came across
when doing some integration work.
Detailed comments about the changes can be found in the bug.
Thanks & best regards