RFR : JDK-8024352 - MBeanOperationInfo accepts any int value as "impact"

[Ujwal Vangapally]

Thanks for the review Mandy,

kindly check if this version is better.

webrev : 
http://cr.openjdk.java.net/~uvangapally/webrev/2017/8024352/webrev.04/

Ujwal


On 11/9/2017 9:10 PM, mandy chung wrote:
>
>
> On 11/9/17 2:40 AM, Ujwal Vangapally wrote:
>> Thanks for the Review Daniel, made changes as suggested.
>>
>> webrev : 
>> http://cr.openjdk.java.net/~uvangapally/webrev/2017/8024352/webrev.03/
>>
>
> Looks good.
>
> Minor comment: in the new test, it can fold some of the println 
> together e.g. line 81 can be merged with line 39 to include the value 
> being passed.  Similarly for the println in the main method.
>
> Mandy
>