highlightjs/highlight.js

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

Open

#3 980 ouverte le 31 janv. 2024

Voir sur GitHub
 (13 commentaires) (0 réactions) (0 assignés)JavaScript (3 497 forks)batch import
buggood first issuehelp welcomelanguage

Métriques du dépôt

Stars
 (22 960 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Le bug se trouve dans le fichier de définition du langage C++ (probablement src/languages/cpp.js). Le motif de mot clé pour `static assert` et mots clés similaires ne doit pas correspondre lorsqu'il est suivi de parenthèses. Regardez comment d'autres mots clés comme `if` sont gérés—ils utilisent probablement un lookahead négatif pour éviter de correspondre à des appels de fonction. La correction consiste à ajuster l'expression régulière pour exclure les parenthèses après le mot clé.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60