bradcypert / bradcypert/httpspec

Investigate memory management issue with HttpRequest URLs in http_client.zig

Abierto
#11 0 comentarios 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Zig
Estrellas
11
Forks
1
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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.
```

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.