Enforce callback signatures with C++ Concepts

Aberta
#1,746 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
52/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Pouca atividade
Stack de tecnologia
cpp
Domínio
api

Direção de pesquisa

Revise as anotações inline de assinatura de #1738 e os pontos de entrada de template listados, incluindo BasicEnv::PostFinalizer, Object::AddFinalizer, External::New, Buffer::New e as APIs de ThreadSafeFunction. Determine as restrições necessárias para cada candidato a Finalizer, Callable, Getter, Setter e Callback. Considera-se concluído quando C++20 Concepts impõem as assinaturas documentadas e produzem diagnósticos no local da chamada.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

blocked SemVer-major

#1738 documented the required signature of the templated finalizer callback parameters as inline comment hints.

These are human-only annotations. The compiler does not enforce them, so a mismatched callable fails deep inside the template instantiation with an unhelpful error.

Once C++20 is the minimum supported standard, we should replace these hints with C++ Concepts that constrain the Finalizer/FinalizerType template parameters. This gives compiler-enforced contracts and clear diagnostics at the call site.

Finalizer candidates:

  • BasicEnv::PostFinalizer
  • Object::AddFinalizer
  • External::New
  • Buffer::New / Buffer::NewOrCopy
  • ThreadSafeFunction::New
  • TypedThreadSafeFunction::New

Callback candidates:

  • Function::New — the JS callback (Callable)
  • PropertyDescriptor::Accessor — Getter / Setter
  • PropertyDescriptor::Function — Callable
  • ThreadSafeFunction::BlockingCall / NonBlockingCall — Callback
  • TypedThreadSafeFunction::BlockingCall / NonBlockingCall — Callback
  • TypedThreadSafeFunction — the CallJs callback
Linguagem predominante
C++
Estrelas
2.4k
Forks
499
Merge médio
2d 11h
PRs com merge (30d)
2

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de nodejs/node-addon-api

Todas as issues de nodejs/node-addon-api

Issues semelhantes

Mais issues de C++

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.