> >  > dladm_delete_conf()     /* delete_conf */
 > >  > dladm_destroy_conf()    /* free_conf */
 > 
 > These two names can be easily confused. 

I agree -- especially since the dladm UI generally has create/delete as
opposites (as per my earlier mail), but here we're correctly using
create/destroy as opposites.

 > Maybe dladm_delete_conf(), dladm_release_conf()?

When I think of "release", I don't think of deallocating something.  What
about keeping destroy, but changing "delete" to "remove"?  Anyway, it's
Dan's call :-)

-- 
meem

Reply via email to