Sundar Yamunachari wrote: > Jan, > * > usr/src/cmd/auto-install/auto_install.c* > > Can you change enable_debug_mode() to set_debug_mode() and > is_debug_mode_enabled() to get_debug_mode()? This will make it more > clear and if want to allow setting more than one debug value in the > future, it will be simpler. >
As it stands right now, the implementation is more correct and more clear for a boolean predicate. If we decide we need multiple levels later we can change it, but I would advise Jan to leave this as-is for now, since the need for multiple levels seems ill-defined. Dave
