bradcypert / bradcypert/httpspec

Investigate memory management issue with HttpRequest URLs in http_client.zig

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

Descrição

Review and refactor how memory is managed for HttpRequest URLs, particularly in test scenarios where the deinit method tries to free global strings. Consider alternative approaches for test setups or memory management.

Reference:
```zig
// TODO: The request de-allocates the memory used by the url in the deinit method.
// Because of this, with this test, we have to dupe it or else it cant deinit a global string.
// Needing to do this is a sign that something is off here, but also newing up requests directly
// isn't really intended either. Either way, need to look into this further.
```

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

Comece com http_client.zig e rastreie a propriedade da URL de HttpRequest durante a inicialização e deinit; depois, inspecione a configuração de testes referenciada. Determine como as strings globais são tratadas e se o workaround de duplicação existente ainda é necessário. Considera-se concluído quando a limpeza da requisição não tenta mais liberar strings globais que não são de sua propriedade e os testes afetados passam.

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

Avaliação

Stack de tecnologia
zig
Domínio
testing
Tipo de issue
Refatoração
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

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