Do we need `wrapper_`?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp, nodejs
- Área
- documentation
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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?
- Lenguaje dominante
- C++
- Estrellas
- 2.6k
- Forks
- 598
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nodejs/node-addon-examples
-
Create Valgrind run of testsAbiertoenhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
nodejs/node-addon-examples#585 · 1 reacción ·
-
Add .d.ts files in samples.Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
nodejs/node-addon-examples#530 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
nodejs/node-addon-examples#445 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 32/100
nodejs/node-addon-examples#444 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
nodejs/node-addon-examples#381 ·
Todos los issues de nodejs/node-addon-examples
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
meshcore-dev/MeshCore#3760 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
deepseek-ai/DeepGEMM-Ascend#29 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
EchoTools/nevr-runtime#2042 ·
Los mantenedores suelen responder en 1 día
-
Download size and speed show scientific notation between 1000 and 1023 unitsPosiblemente ocupada @RaphaelFakhri la tomó hoy. Abierto
Dificultad 2/5 Menos de una hora Aptitud para principiantes 88/100
kiwix/kiwix-desktop#1570 ·
Los mantenedores suelen responder en 1 día
-
Device Request
Dificultad 2/5 Medio día Aptitud para principiantes 62/100
dresden-elektronik/deconz-rest-plugin#8686 ·
Los mantenedores suelen responder en 1 día