il_states.c: 2 * bad if test ?

A full featured cross-platform image library

Brought to you by: doomwiz

Status: open

Owner: nobody

Labels: None

Priority: 5

Updated: 2013-06-22

Created: 2013-06-22

Creator: dcb

Private: No

I just ran the static analysis checker "cppcheck"
over the source code of DevIL-1.7.8

It said

[../src-IL/src/il_states.c:1127]: (warning) Logical disjunction always evaluates to true: Param >= 0 || Param <= 2.

[../src-IL/src/il_states.c:1133]: (warning) Logical disjunction always evaluates to true: Param >= -1 || Param <= 255.

In both cases, suggest replace || with &&