cvs commit: src/sys/boot/powerpc/loader conf.c
Peter Grehan
grehan at freebsd.org
Thu Dec 11 22:52:26 PST 2003
> These changes seem to be basically correct and have correct style
> except for not merging the declarations of most of the new "int ch"
> variables with the declarations of the other int variables.
OK, I'll sweep through and fix that.
> The fix for kg_nice in proc.h is missing. Only the relatively unimportant
> declaration of ki_nice in user.h is fixed.
Yep, mis-edited that one out of the diff :-(.
> What bugs does putting negative niceness in the (unsigned) char variable
> kg_nice cause?
Comparisons to NZERO don't work as expected e.g.
/usr/home/grehan/freebsd/dev/src/bin/ps/print.c: In function `state':
/usr/home/grehan/freebsd/dev/src/bin/ps/print.c:250: warning: comparison
is always false due to limited range of data type
later,
Peter.
More information about the cvs-src
mailing list