adopted-ember-addons / adopted-ember-addons/ember-href-to

Possible memory leak.

Aberta
#8 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
JavaScript
Estrelas
0
Forks
1
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

Hello! I noticed the original repo closed all the issues and PRs, and so im just cross posting the issue... PR.... isnt this a memory leak?

https://github.com/adopted-ember-addons/ember-href-to/blob/c480104661c5150f3f46d8d0642c46bffff89849/addon/helpers/href-to.js#L35

https://github.com/intercom/ember-href-to/pull/177/changes

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Inspect the linked `addon/helpers/href-to.js` (line 35 in the referenced commit) in this repository and compare it with `intercom/ember-href-to/pull/177/changes` to spot the possible leak pattern. Then read the surrounding helper code to trace object creation and cleanup paths. Start with the helper implementation first, then verify a fix is complete by checking that repeated `href-to` usage no longer appears to retain unnecessary memory.

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

Avaliação

Stack de tecnologia
javascript
Domínio
frontend
Tipo de issue
Bug
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Pouca atividade
Clareza
Razoavelmente clara
Facilidade para iniciantes
58/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.