Debug and fix memory leaks

Abierto
#20 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
25/100
Tipo de issue
Error
Claridad
Necesita aclaración
Estado de actividad
Estancado
Stack tecnológico
android, cpp, java, javascript
Área
mobile-dev, performance

Línea de trabajo

En el issue no se nombra ningún archivo ni test. Empieza revisando el commit referenciado y, después, investiga la fuga restante de Java Interface Implementation en el lado de JS; se considera terminado cuando la fuga indicada está corregida y se ha recuperado la memoria.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

New runtimes work differently and are not well tested yet for memory leaks. We are working on fixing issues anything we find.

  • When a java object is finally freed in Java, we notify the JS side to free the JS counterpart. At this time, the Java object is held weakly in Java too using a WeakReference. While the Java object is weakly held, the WeakReference is strongly kept in a HashMap never getting removed. In this PR we release the WeakReferences too on GC event ensuring that all memory is reclaimed. Fixed in https://github.com/NativeScript/napi-android/commit/5e48302fc0eeab63eae6935e864de36d25100e2c
  • Investigate Java Interface Implementation leaking on JS Side
Lenguaje dominante
C++
Estrellas
33
Forks
3
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de NativeScript/napi-android

Todos los issues de NativeScript/napi-android

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.