Do we need `wrapper_`?

Ouverte
#140 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
35/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
cpp, nodejs
Domaine
documentation

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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?

Langage dominant
C++
Étoiles
2.6k
Forks
602
Aucune PR mergée en 30 j

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de nodejs/node-addon-examples

Toutes les issues de nodejs/node-addon-examples

Issues similaires

Plus d'issues C++

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.