diff options
author | ray <ray@FreeBSD.org> | 2012-09-04 23:16:55 +0000 |
---|---|---|
committer | ray <ray@FreeBSD.org> | 2012-09-04 23:16:55 +0000 |
commit | f2d384bbd0df5897a4b1f48f872d443dc077957c (patch) | |
tree | d388a48103e16388e4a2ccd57c0a8aaa4986ecce /sys/kern/subr_hints.c | |
parent | 374ac87db9c23b766435d587bce64848473d6fff (diff) | |
download | FreeBSD-src-f2d384bbd0df5897a4b1f48f872d443dc077957c.zip FreeBSD-src-f2d384bbd0df5897a4b1f48f872d443dc077957c.tar.gz |
Style fixes.
Suggested by: mdf
Approved by: adrian (menthor)
Diffstat (limited to 'sys/kern/subr_hints.c')
-rw-r--r-- | sys/kern/subr_hints.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/sys/kern/subr_hints.c b/sys/kern/subr_hints.c index b2fe0bf..db45fb8 100644 --- a/sys/kern/subr_hints.c +++ b/sys/kern/subr_hints.c @@ -31,8 +31,8 @@ __FBSDID("$FreeBSD$"); #include <sys/lock.h> #include <sys/malloc.h> #include <sys/mutex.h> -#include <sys/systm.h> #include <sys/sysctl.h> +#include <sys/systm.h> #include <sys/bus.h> /* @@ -52,9 +52,9 @@ static char *hintp; static int sysctl_hintmode(SYSCTL_HANDLER_ARGS) { - int error, i, from_kenv, value, eqidx; const char *cp; char *line, *eq; + int eqidx, error, from_kenv, i, value; from_kenv = 0; cp = kern_envp; @@ -62,7 +62,7 @@ sysctl_hintmode(SYSCTL_HANDLER_ARGS) /* Fetch candidate for new hintmode value */ error = sysctl_handle_int(oidp, &value, 0, req); - if (error || !req->newptr) + if (error || req->newptr == NULL) return (error); if (value != 2) @@ -73,8 +73,11 @@ sysctl_hintmode(SYSCTL_HANDLER_ARGS) switch (hintmode) { case 0: if (dynamic_kenv) { - /* Already here */ - hintmode = value; /* XXX: Need we switch or not ? */ + /* + * Already here. But assign hintmode to 2, to not + * check it in the future. + */ + hintmode = 2; return (0); } from_kenv = 1; @@ -98,7 +101,7 @@ sysctl_hintmode(SYSCTL_HANDLER_ARGS) continue; } eq = strchr(cp, '='); - if (!eq) + if (eq == NULL) /* Bad hint value */ continue; eqidx = eq - cp; |