I found these very serious warnings that appear to be from the recent
event changes.  Are the parameters genuinely intended to be uint, in
which case they don't need a ">= 0" test?  Also with the "< 0" tests
in csinput.cpp shouldn't the code do something smarter than just
return like log an error?

Michael D. Adams
[EMAIL PROTECTED]

/src/CrystalSpace/CS-dev/./include/csutil/csinput.h:269: warning:
comparison of unsigned expression >= 0 is always true
/src/CrystalSpace/CS-dev/./include/csutil/csinput.h:348: warning:
comparison of unsigned expression >= 0 is always true
/src/CrystalSpace/CS-dev/libs/csutil/csinput.cpp:635: warning:
comparison of unsigned expression < 0 is always false
/src/CrystalSpace/CS-dev/libs/csutil/csinput.cpp:714: warning:
comparison of unsigned expression < 0 is always false
/src/CrystalSpace/CS-dev/libs/csutil/csinput.cpp:798: warning:
comparison of unsigned expression < 0 is always false
/src/CrystalSpace/CS-dev/libs/csutil/csinput.cpp:842: warning:
comparison of unsigned expression < 0 is always false
/src/CrystalSpace/CS-dev/libs/csutil/event.cpp:204: warning: comparison of unsig
ned expression >= 0 is always true
/src/CrystalSpace/CS-dev/libs/csutil/event.cpp:325: warning: comparison of unsig
ned expression >= 0 is always true


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_idv37&alloc_id865&op=click
_______________________________________________
Crystal-main mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/crystal-main
Unsubscribe: mailto:[EMAIL PROTECTED]

Reply via email to