[PD-dev] fiddle~.c warning

Miller Puckette msp at ucsd.edu
Fri Apr 1 20:25:59 CEST 2016


These are certainly bugs :)

M

On Fri, Apr 01, 2016 at 03:41:43AM -0600, Dan Wilcox wrote:
> Curious about a couple clang warnings for fiddle~.c:
> 
> pure-data/extra/fiddle~/fiddle~.c:1120:20: warning: comparison of constant 0 with boolean expression is always false [-Wtautological-constant-out-of-range-compare]
>     if (!npeakanal < 0) npeakanal = 0;
>         ~~~~~~~~~~ ^ ~
> pure-data/extra/fiddle~/fiddle~.c:1122:19: warning: comparison of constant 0 with boolean expression is always false [-Wtautological-constant-out-of-range-compare]
>     if (!npeakout < 0) npeakout = 0;
>         ~~~~~~~~~ ^ ~
> 
> Since both npeakanal & npeakout are longs, the ! operator is basically checking if they are 0 right by converting to a bool. I follow that. I’m just not sure how a bool can be less than 0 and I guess clang isn’t either :)
> 
> --------
> Dan Wilcox
> @danomatika <https://twitter.com/danomatika>
> danomatika.com <http://danomatika.com/>
> robotcowboy.com <http://robotcowboy.com/>

> _______________________________________________
> Pd-dev mailing list
> Pd-dev at lists.iem.at
> http://lists.puredata.info/listinfo/pd-dev




More information about the Pd-dev mailing list