This is a discussion on Re: In function `k_getuid6': comparison is always false due to limited range of data type within the mailing.openbsd.tech forums, part of the OpenBSD category; --> On Wed, Jun 13, 2007 at 12:45:05AM +0200, Stefan Sperling wrote: > I just saw this while compiling -current: ...
| |||||||
| Register | FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read |
| ||||
| On Wed, Jun 13, 2007 at 12:45:05AM +0200, Stefan Sperling wrote: > I just saw this while compiling -current: > > ===> libexec/identd > cc -O2 -pipe -Wall -c /usr/src/libexec/identd/identd.c > cc -O2 -pipe -Wall -c /usr/src/libexec/identd/openbsd.c > /usr/src/libexec/identd/openbsd.c: In function `k_getuid6': > /usr/src/libexec/identd/openbsd.c:89: warning: comparison is always > false due to limited range of data type > /usr/src/libexec/identd/openbsd.c:92: warning: comparison is always > false due to limited range of data type > These are gcc warnings that can easily win the most stupid gcc warnings award. So somebody adds a currently superfluous check because he is conservative, now somebody sees these warnings and removes all the checks. A bit later a third person changes the data type to a larger/signed one and bam we have a problem. Just ignore the warnings. -- :wq Claudio |
| Thread Tools | |
| Display Modes | |
|
|