redundant code gone from is_number

the "|| return 1" in "f() { command || return 1; }" is only
useful as a way to coalesce all failures to a single exit code,
and i don't care about that.
man page gives more info on config files