Investigate memory management issue with HttpRequest URLs in http_client.zig
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- zig
- Domaine
- testing
Piste de recherche
Commencez par http_client.zig et suivez la propriété de l’URL de HttpRequest lors de l’initialisation et deinit, puis examinez la configuration de test référencée. Déterminez comment les chaînes globales sont gérées et si le workaround de duplication existant est toujours nécessaire. C’est terminé lorsque le nettoyage de la requête n’essaie plus de libérer des chaînes globales qui ne lui appartiennent pas et que les tests concernés réussissent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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:
// 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.
- Langage dominant
- Zig
- Étoiles
- 11
- Forks
- 1
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de bradcypert/httpspec
-
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
bradcypert/httpspec#22 · 1 commentaire ·
-
enhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
bradcypert/httpspec#12 ·
Toutes les issues de bradcypert/httpspec
Issues similaires
-
agent/security hive/hosted-available-lke648397-260827-5n31 security
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
jbaruch/coding-policy#497 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
test-change-proposal
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
web-platform-tests/interop#1455 ·
-
.NET Flaky Test Testing Tests
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
getsentry/sentry-dotnet#5617 · 1 commentaire ·