Darren,
On 10/25/11 3:13 AM, Darren Kenny wrote:
Hi Drew,
I took at look at this, in general all looking good, but I do have some
questions/comments:
- Generally
I wonder if the name 'create_child_slice' doesn't convey the fact that
all existing slices will be deleted well enough.
I don't have any specific suggestion and
"create_slice_replacing_all_others" is probably a bit verbose ;)
If you feel strongly about the name, I could maybe do:
create_single_slice() or something like that?
- test_shadow_list.py
Would it be worth adding a test to verify that it's the only slice
created after the call.
Sure, I could add that. I'll get a code review up to you shortly.
-Drew
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
