Hi Claes,

This looks fine to me.

Thanks,
Jini. (Not a Reviewer)

On 4/30/2018 3:00 PM, Claes Redestad wrote:
Hi,

ClhsdbJhisto has a hard-coded dependency on an ImmutableCollections$SetN$1 appearing in the jhisto output, which is fragile and sometimes break after JDK-8201650. I suggest removing it from the test:

diff -r 96d4658eb7f2 test/hotspot/jtreg/serviceability/sa/ClhsdbJhisto.java
--- a/test/hotspot/jtreg/serviceability/sa/ClhsdbJhisto.java  Mon Apr 30 09:15:44 2018 +0200 +++ b/test/hotspot/jtreg/serviceability/sa/ClhsdbJhisto.java  Mon Apr 30 11:28:08 2018 +0200
@@ -62,8 +62,7 @@
                      "java.nio.HeapByteBuffer",
                      "java.net.URI",
                      "LingeredAppWithInterface",
-                    "ParselTongue",
-                    "ImmutableCollections$SetN$1"));
+                    "ParselTongue"));

              test.run(theApp.getPid(), cmds, expStrMap, null);
          } catch (Exception ex) {

Reply via email to