Microsoft/monaco-editor

C++ line continuation mark (\) should be ignored everywhere

Open

#1.184 geöffnet am 14. Nov. 2018

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.283 Forks)batch import
feature-requestgrammarshelp wantedopen for PR

Repository-Metriken

Stars
 (14.836 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6h 26m) (15 gemergte PRs in 30 T)

Beschreibung

monaco-editor version: 0.15.1 Browser: Chrome (maybe others) OS: Windows

In C/C++ \ is used as line continuation mark so defining macros in multi-line is possible:

#define MACRO do { \
} while(false);

It is also possible to change a single line comment to multi-line with \:

// Multi \
   Line \
   Comment

As you see in above snippet, Github syntax highlighter accept this syntax in comments and colorize line 2 and 3 as comments, but Monaco does not:

image

This is somehow strange that the VSCode highlights it correctly but Monaco does not. image

Contributor Guide

Research-Richtung
Das Problem ist, dass der C++ Zeilentrennungs Backslash in Kommentaren und Zeichenketten nicht ignoriert wird. Die Lösung erfordert wahrscheinlich eine Änderung der von Monaco verwendeten C++ TextMate Grammatikdatei (z. B. `src/basic languages/cpp/cpp.tmLanguage.ts`). Die Grammatik muss den Backslash als Zeilentrennungszeichen in Makros und einzeiligen Kommentaren behandeln. Schauen Sie sich an, wie VSCode damit umgeht (da VSCode Monaco verwendet, aber korrektes Verhalten zeigt), und übernehmen Sie diese Logik. Die Änderung ist auf die Grammatikdefinition beschränkt.
Tech Stack
javascripttypescript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65