Review request for JMC-6408: Cleanup of MBeanOperationsWrapper

[Marcus Hirt]

You're quite right! I'll fix the order in a follow up check-in and 
set you as reviewer, unless someone objects.

Kind regards,
Marcus

On 2019-02-13, 15:20, "Langer, Christoph" <christoph.langer at sap.com> wrote:

    Hi Marcus,
    
    looks good to me. Nice cleanup.
    
    However, I'd suggest another order of arguments for MBeanOperationWrapper:: createOperations, like this:
    
    static Collection<IOperation> createOperations(MBeanServerConnection connection , ObjectName objectName, MBeanOperationInfo[] operations)
    
    this would match the order of arguments of the constructor of MBeanOperationWrapper.
    
    Best regards
    Christoph
    
    > -----Original Message-----
    > From: jmc-dev <jmc-dev-bounces at openjdk.java.net> On Behalf Of Marcus
    > Hirt
    > Sent: Mittwoch, 13. Februar 2019 10:10
    > To: jmc-dev at openjdk.java.net
    > Subject: Review request for JMC-6408: Cleanup of
    > MBeanOperationsWrapper
    > 
    > Hi all,
    > 
    > Please review this fix to clean up the MBeanOperationsWrapper a bit
    > (should not subclass ArrayList).
    > 
    > Jira:   https://bugs.openjdk.java.net/browse/JMC-6408
    > Webrev: http://cr.openjdk.java.net/~hirt/JMC-6408/webrev.0/
    > 
    > Kind regards,
    > Marcus
    >