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