github / github/codeql

LGTM.com - false positive - py/unreachable-statement

Ouverte
#6,848 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
false-positive
Langage dominant
CodeQL
Étoiles
10.1k
Forks
2.1k
Merge moyen
2 j 15 h
PR mergées (30 j)
141

Description

**Description of the false positive**

LGTM gives an alert about unreachable code, but this code *is* reachable and I can easily trigger the print statement on the line LGTM says is unreachable, by running it as
```sh
touch a.c
submit -u https://team:team@www.domjudge.org/demoweb -c nwerc18 a.c
```

I don't have much experience with Python and this code was written by another developer in our project. My only hunch is that the global `num_warnings` variable is imported in the local scope in `warn_user()` and maybe this is not detected properly?

**URL to the alert on the project page on LGTM.com**

https://lgtm.com/projects/g/DOMjudge/domjudge/snapshot/2f8248424c1572f48216d68a451004e6e2d30d00/files/submit/submit?sort=name&dir=ASC&mode=heatmap#x6c8458ef33745fdc:1

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.