Apologies for taking over here ... This test is maintained externally in JSR166 CVS. We prefer a version that works well with both old and new outputs. I suggest instead
"(?m)^ *[0-9]+: +([0-9]+) +[0-9]+ +\\Q"+className+"\\E(?:$| )"; Does that work with latest serviceability changes? On Fri, Apr 29, 2016 at 11:00 AM, harold seigel <harold.sei...@oracle.com> wrote: > Hi, > > Please review this extra small fix for test TimedAcquireLeak.java. The test > was broken by the recent change to add module information to the JMAP class > histogram. This fixes the test's regular expression to handle the possible > additional module information. > > Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8155727/ > > JBS bug: https://bugs.openjdk.java.net/browse/JDK-8155727 > > Thanks! Harold >
