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

Mandy Chung mandy.chung at oracle.com
Fri Nov 8 15:48:26 UTC 2013


Thanks you all for the review.

I'll rename AbstractOperatingSystemImpl before I push.

Mandy

On 11/8/2013 5:22 AM, Alan Bateman wrote:
> On 08/11/2013 08:40, Jaroslav Bachorik wrote:
>>
>> AbstractOperatingSystemImpl should be an abstract class as its name 
>> already indicates.
> Right, it probably should be something like BaseOperatingSystemImpl.
>
> Otherwise it looks good to me too.
>
> -Alan.




More information about the build-dev mailing list