LGTM!

-Drew

On 6/21/11 8:48 AM, Darren Kenny wrote:
Hi,

Could I please get some eyes to look over the simple change for:

    7057322 DataObject root property clashes with attribute name already in
            use by sub-classes

the webrev is at:

    http://cr.opensolaris.org/~dkenny/bug_7057322/

Basically, I looked around to see what was using 'root' in DataObject
sub-classes, and there are several.

I then looked to see if there were any using root_object, which probably more
correctly describes what it is we're referencing, and there was nothing.

So I've replaced any instances of '.root' with '.root_object' where appropriate
(there really is only one consumer of it right now).

I then re-ran the unit tests associated, and things worked as expected.

Thanks,

Darren.
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to