Looks good! Thanks, /Staffan
> On 3 dec 2014, at 23:06, KEVIN WALLS <[email protected]> wrote: > > Hi, > > This is a review request for a changing a couple of characters in a test. > It's the same test I just changed with another review, I kept it separate so > as to complicate the review already in progress (it had gone on long > enough..). > > > The test contains an embedded utf8 character, to test a jmap/SA heap dumping > fix. The test is not broken here in 9/hs-rt, but in another clone I saw this > character get corrupted by a merge: it was rewritten as literal ?? question > marks. It wasn't clear if it was an hg issue (I couldn't reproduce that) or > perhaps more likely the editor used to do a merge. > > To avoid the same thing happening again we can change the utf8 char to \uXXXX > format. > > The change below I tested to check the test still correctly fails on a > non-fixed JVM, and passes on a fixed one > > bug: > https://bugs.openjdk.java.net/browse/JDK-8061785 > > change: > $ hg diff test/serviceability/sa/jmap-hashcode/Test8028623.java > diff --git a/test/serviceability/sa/jmap-hashcode/Test8028623.java > b/test/serviceability/sa/jmap-hashcode/Test8028623.java > --- a/test/serviceability/sa/jmap-hashcode/Test8028623.java > +++ b/test/serviceability/sa/jmap-hashcode/Test8028623.java > @@ -41,12 +41,12 @@ > > public class Test8028623 { > > - public static int à = 1; > + public static int \u00CB = 1; > public static String dumpFile = "heap.out"; > > public static void main (String[] args) { > > - System.out.println(Ã); > + System.out.println(\u00CB); > > try { > if (!Platform.shouldSAAttach()) { > > ---------------------------------------------------------------------------------------------- > > Thanks > Kevin >
