Thank you Chris. On Wed, Jun 5, 2019, 19:22 Chris Plummer <chris.plum...@oracle.com> wrote:
> Looks good. Thanks for fixing quickly. > > Chris > > On 6/5/19 9:50 AM, Thomas Stüfe wrote: > > Hi all, > > may I have a small review please for this patch: > > cr: > http://cr.openjdk.java.net/~stuefe/webrevs/8225348-eventstest-failed/webrev.00/webrev/ > bug: https://bugs.openjdk.java.net/browse/JDK-8225348 > > JDK-8224600 introduced a new jcmd, VM.events, and an associated test. > > In run_selected(CommandExecutor executor), I want to make sure the > "Events" log is not printed and therefore scan for "Events" on stdout, but > that term is too broad and can be part of normal output, giving a false > positive. > > Fix is to make the pattern matching more strict. > > Thanks, Thomas > > >
