search

jgigault / 42FileChecker

42FileChecker is a tiny bash script developed at 42 school for testing and checking files according to the rules of the subjects
Apache License 2.0
305 stars 56 forks source link

Leaks #124

Open cgarrot opened 6 years ago

cgarrot commented 6 years ago

il faut free line dans les main de test de ./src/gnl

jgigault commented 5 years ago

Salut @ripZeide Pourrais-tu décrire ton propos et le comportement que tu t'attend à voir ? La raison pour laquelle il n'y a pas d'appel à "free" dans les tests est que cela empêche le choix de certains étudiants de gérer la mémoire directement dans get_next_line et de libérer la mémoire de la dernière ligne en dehors de la boucle. À mon sens c'est une mauvaise option mais le sujet de l'école ne décrit pas le comportement exact attendu. J'ai préparé une PR (#132) pour ajouter un test à titre d'information, qu'en penses-tu ? Quelle est ton opinion ?