On Mon, Mar 17, 2003 at 09:11:12AM -0800, David O'Brien wrote:
> On Mon, Mar 17, 2003 at 12:28:34AM -0800, Crist J. Clark wrote:
> > +kldxref_start () {
> > + if [ -z "$kldxref_module_path" ]; then
> > + MODULE_PATHS=`sysctl -n kern.module_path`
> > + else
> > + MODULE_PATHS="$kldxref_module_path"
> > + fi
>
> Please change the logic to positive logic:
>
> if [ -n "$kldxref_module_path" ]; then
> MODULE_PATHS="$kldxref_module_path"
> else
> MODULE_PATHS=`sysctl -n kern.module_path`
> fi
Is there a technical reason for that or is it just a "style" issue?
--
Crist J. Clark | [EMAIL PROTECTED]
| [EMAIL PROTECTED]
http://people.freebsd.org/~cjc/ | [EMAIL PROTECTED]
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
