Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core]: Use weak references for node subscriptors #10

Open
redfox-mx opened this issue Aug 17, 2024 · 0 comments
Open

[core]: Use weak references for node subscriptors #10

redfox-mx opened this issue Aug 17, 2024 · 0 comments

Comments

@redfox-mx
Copy link

redfox-mx commented Aug 17, 2024

Muy buenas tardes equipo, hace tiempo que he notado que la aplicación empieza consumir recursos con forme se va navegando en la aplicación, si bien, esto puede ser normal al inspeccionar el código me he dado cuenta que se tiene referencias fuertes a los subscriptors lo cual imposibilita al GC liberar las referencias a elementos que ya fueron borrados del DOM

/**
* The map of subscriptors.
*
* @type {Map<WCNode, Subscriptor>}
*/
this.subscriptors = new Map();

Esto podria ser solucionado con el uso de referencias debiles, de tal forma que una vez que el bridge ha removido una page del dom el GC pueda recolectar y liberar correctamente estos recursos.

Para ello, podemos hacer uso de WeakMap de tal forma que quitemos esa referencia fuerte al nodo del DOM y pueda ser recolectado, de la misma forma, podemos implementar futuras optimizaciones, como la limpieza de las subscripciones cuando el nodo del dom ya no es necesario y no se debe emitir mas valores a dicho nodo

atte: diegojesus.hernandez.gonzalez@bbva.com

@redfox-mx redfox-mx changed the title [fix] Use weak references for node subscriptors [core]: Use weak references for node subscriptors Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant