Do we need `wrapper_`?

Aberta
#140 3 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
35/100
Tipo de issue
Refatoração
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
cpp, nodejs
Domínio
documentation

Direção de pesquisa

Start by searching the ObjectWrap N-API examples for wrapper_, napi_wrap(), and napi_delete_reference(). Confirm whether wrapper_ is unused and check how the examples build; done means removing unnecessary storage without breaking the examples or their illustration of object wrapping.

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

Descrição

In our ObjectWrap N-API examples we store the napi_ref returned from napi_wrap() in the native object instance although we never use it. To napi_delete_reference() in the destructor we need to also store the napi_env on the native instance – a practice we discourage.

Do we do this for illustration purposes? Can we remove this, since wrapper_ is not being used anywhere?

Linguagem predominante
C++
Estrelas
2.6k
Forks
602
Nenhum PR com merge em 30d

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-examples

Todas as issues de nodejs/node-addon-examples

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.