Review request for 8025985: com.sun.management.OSMBeanFactory should not be public

[Mandy Chung]

On 11/10/2013 12:30 PM, Alan Bateman wrote:
> On 08/11/2013 15:48, Mandy Chung wrote:
>> Thanks you all for the review.
>>
>> I'll rename AbstractOperatingSystemImpl before I push.
>>
>> Mandy
>
> I think there is a problem with the change that was pushed. The 
> Windows implementation of OperatingSystemImpl now implements 
> java.lang.management.OperatingSystemMXBean whereas it actually 
> implements com.sun.management.OperatingSystemMXBean. This causes a 
> number of management tests to fail (only on Windows). If you agree 
> then I'd like to fix it up with the attached patch to get the tests 
> passing on Windows again.

Thanks for fixing it.  The patch looks good.

Mandy

>
> -Alan.
>
>
> diff --git 
> a/src/windows/classes/sun/management/OperatingSystemImpl.java 
> b/src/windows/classes/sun/management/OperatingSystemImpl.java
> --- a/src/windows/classes/sun/management/OperatingSystemImpl.java
> +++ b/src/windows/classes/sun/management/OperatingSystemImpl.java
> @@ -25,7 +25,7 @@
>
>  package sun.management;
>
> -import java.lang.management.OperatingSystemMXBean;
> +import com.sun.management.OperatingSystemMXBean;
>
>  /**
>   * Implementation class for the operating system.