highlightjs/highlight.js

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

Open

#3.980 aberto em 31 de jan. de 2024

Ver no GitHub
 (13 comments) (0 reactions) (0 assignees)JavaScript (3.497 forks)batch import
buggood first issuehelp welcomelanguage

Métricas do repositório

Stars
 (22.960 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
O bug está no arquivo de definição da linguagem C++ (provavelmente src/languages/cpp.js). O padrão de palavra chave para `static assert` e palavras chave semelhantes não deve corresponder quando seguido por parênteses. Veja como outras palavras chave como `if` são tratadas—elas provavelmente usam uma verificação negativa (negative lookahead) para evitar corresponder a chamadas de função. A correção envolve ajustar a expressão regular para excluir parênteses após a palavra chave.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60