[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    RFR 8058089: api/javax_management/loading/MLetArgsSupport.html\#MLetArgsSupportTest0001
From:       Jaroslav Bachorik <jaroslav.bachorik () oracle ! com>
Date:       2014-09-11 9:54:58
Message-ID: 54117172.5040700 () oracle ! com
[Download RAW message or body]

Please, review the following regression fix

Issue  : https://bugs.openjdk.java.net/browse/JDK-8058089
Webrev : http://cr.openjdk.java.net/~jbachorik/8058089/webrev.00

The regression was introduced by an en-mass update of new 
`Integer(param)` to `Integer.valueOf(param)`. For some reason in the 
MLet code only `param` was used instead of `Integer.valueOf(param)`. 
I've fixed this problem and also took the liberty to convert new Float() 
and new Double() to .valueOf(...) form. I also added a reg test 
asserting the correctness of param conversions performed by the MLet class.


Thanks,

-JB-
[prev in list] [next in list] [prev in thread] [next in thread]