Menu

#10 flasm.c:520: bad if test ?

v1.0_(example)
open
nobody
None
5
2014-04-24
2014-04-24
dcb
No

[flasm.c:520]: (warning) Logical disjunction always evaluates to true: numActions > 0 || numActions < 256.

Source code is

if (numActions > 0 || numActions < 256)
    output[len - back - 1] = (byte) numActions;

Suggest new code

if (numActions > 0 && numActions < 256)
    output[len - back - 1] = (byte) numActions;

Discussion


Log in to post a comment.