github / github/codeql

LGTM.com - false positive: constant-comparison with assignment on one side

Offen
#3,496 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
C++ false-positive
Vorherrschende Sprache
CodeQL
Sterne
10.1k
Forks
2.1k
Ø Merge
2 T. 15 Std.
Gemergte PRs (30 T.)
141

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.