LGTM.com - false positive: constant-comparison with assignment on one side
Abierto
C++
false-positive
- Lenguaje dominante
- CodeQL
- Estrellas
- 10.1k
- Forks
- 2.1k
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 141
Descripción
I have
int next_char;
...
(next_char = gzgetc(file_)) != -1
where gzgetc is from zlib:
ZEXTERN int ZEXPORT gzgetc OF((gzFile file));
It triggers constant-comparison rule with message:
> Comparison is always true because ... = ... >= 0.
In this case (a = b) is integer assignment and can be negative.
https://lgtm.com/projects/g/project-gemmi/gemmi/snapshot/d1a210cf4810719b905806a690feade01e6eb5c9/files/include/gemmi/gz.hpp?sort=name&dir=ASC&mode=heatmap#x4ec8a4d86c53b859:1
Guía de contribución
Evaluación
Este issue todavía no se ha evaluado.