Quick-eval targets ending in a comment are deemed invalid

Offen

#743 erstellt am 02.2.2021 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 48/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: typescript, vscode
Bereich: developer-experience, tooling

Rechercherichtung

Reproduziere quick-eval in der VS Code CodeQL-Erweiterung mit dem bereitgestellten QL-Snippet, indem du die Zeile auswählst, die mit einem Kommentar endet. Verfolge den Einstiegspunkt von quick-eval und die Verarbeitung der Auswahl. Überprüfe anschließend, dass Kommentare am Ende einer Auswahl ignoriert werden, während Kommentare innerhalb der Auswahl unverändert bleiben.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug VSCode

Describe the bug
If the piece of code selected for quick eval ends in a comment (// or /* ... */), then quick-eval fails. This is especially troublesome if one is selecting a subset of lines using the left margin, and the last of these happens to have a comment attached.

Comments inside the selected block are unproblematic. The problem only arises at the end of a selection.

To Reproduce
Put the following code in a suitable QL buffer. Select the line starting with result and do a quick-eval.

int getRandomNumber() {
  result = 4 // chosen by fair dice roll. guaranteed to be random.
}

Expected behavior
The comment is ignored, and quick-eval executes successfully

Additional context
Seen using CodeQL extension version 1.3.10 and CLI version 2.4.2+202102021519plus (built locally)

Vorherrschende Sprache: TypeScript
Sterne: 539
Forks: 240
Ø Merge: 2 T. 18 Std.
Gemergte PRs (30 T.): 37

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus github/vscode-codeql

`createLockFileForStandardQuery` always creates temp lock because `packPacklist` doesn't return lock files Offen

bug VSCode

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

github/vscode-codeql#4471 · 1 Kommentar · Erstellt am 22.7.2026
Add ability to use contextual queries to find references when already on a reference Offen

bug enhancement VSCode

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

github/vscode-codeql#592 · 2 Kommentare · Erstellt am 27.9.2020
Allow reloading database contents from the UI Offen

enhancement VSCode

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100

github/vscode-codeql#4432 · 1 Kommentar · Erstellt am 20.6.2026
"Add Database Source to Workspace" does not preserve loaded database Offen

bug VSCode

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100

github/vscode-codeql#4356 · Erstellt am 03.4.2026
"Show AST" not working in codeql tab Offen

bug VSCode

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 64/100

github/vscode-codeql#4339 · 1 Kommentar · Erstellt am 22.3.2026

Alle Issues in github/vscode-codeql