JMC-5639: Can't resolve event type names without Recording Settings event

Marcus Hirt marcus.hirt at oracle.com
Thu Sep 20 16:05:11 UTC 2018


Thanks gents. The change has been pushed!

Kind regards,
Marcus

On 2018-09-20, 08:35, "jmc-dev on behalf of Miro Wengner" <jmc-dev-bounces at openjdk.java.net on behalf of miro.wengner at gmail.com> wrote:

    The patch looks good to me, I’ve it tested and I go proper messages
    Cheers,
    Miro
    
    > On Sep 20, 2018, at 5:25 PM, Mario Torre <neugens at redhat.com> wrote:
    > 
    > On 09/20/2018 04:29 PM, Marcus Hirt wrote:
    >> Took a closer look at this just now, and realized that there is one additional
    >> 
    >> change required:
    >> 
    >> 
    >> 
    >> http://cr.openjdk.java.net/~hirt/JMC-5663/webrev.03/
    > 
    > The patch looks good, I would like a test added though.
    > 
    > Perhaps, that could be the start of a shared repository of records for
    > tests, like we discussed over IRC?
    > 
    > Cheers,
    > Mario
    > 
    > --
    > Mario Torre
    > Associate Manager, Software Engineering
    > Red Hat GmbH <https://www.redhat.com>
    > 9704 A60C B4BE A8B8 0F30  9205 5D7E 4952 3F65 7898
    
    
    




More information about the jmc-dev mailing list