highlightjs/highlight.js

(cpp) Keywords followed by parens are treated as `built_in`

Open

#3980 aperta il 31 gen 2024

Vedi su GitHub
 (13 commenti) (0 reazioni) (0 assegnatari)JavaScript (3497 fork)batch import
buggood first issuehelp welcomelanguage

Metriche repository

Star
 (22.960 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Describe the issue Keywords, such as static_assert, sizeof, etc. are typically followed by parentheses. static_assert; will be highlighted as if it's a keyword, but static_assert(true); is highlighted as a built_in.

image

Which language seems to have the issue?

<code class="hljs cpp">
<code class="hljs c++">

Are you using highlight or highlightAuto?

highlight, I think?

Sample Code to Reproduce

In reveal.js, with a freshly-downloaded hljs, add the following:

<pre><code class="hljs c++" data-trim data-noescape data-line-numbers style="overflow: visible;">
  static_assert; // highlighted as keyword
  static_assert(true); // highlighted as built_in
</code></pre>

Expected behavior

Keywords continue to use their keyword colour when followed by parens.

Guida contributor

Direzione di ricerca
Il bug si trova nel file di definizione del linguaggio C++ (probabilmente src/languages/cpp.js). Il pattern della parola chiave per `static assert` e parole chiave simili non dovrebbe corrispondere quando è seguito da parentesi. Osserva come vengono gestite altre parole chiave come `if`—probabilmente usano un lookahead negativo per evitare di corrispondere a chiamate di funzione. La correzione comporta la modifica dell'espressione regolare per escludere le parentesi dopo la parola chiave.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60