search

alelievr / libft-unit-test

457 stars 88 forks source link

atof test #43

Closed Karocyt closed 6 years ago

Karocyt commented 6 years ago

Tu compare le résultat à l'original, mais par définition, le NaN n'est égal à rien, même pas à lui-même, donc le test semble impossible par conception.

(nan == nan) -> false isnan(nan) -> true

Des bisous et merci pour tout !

alelievr commented 6 years ago

Salut, merci pour cette issue :)

Le test est fix dans la dernière version de master.

Karocyt commented 6 years ago

Merci, je viens de voir le [OK] en correction ! =D Deux autres bricoles tant qu'à faire, mon ancien strsplit passait tes tests en écrasant le dernier élément de l'array dans certains cas ( ++ manquant avant l'ajout du Null), et mon lstmap répondrait true à une comparaison d'adresse du premier élément (va savoir pourquoi, je mets mon pointeur au premier élément, je ne modifie que les suivants)

Le sam. 21 avr. 2018 à 14:31, Antoine Lelievre notifications@github.com a écrit :

Salut, merci pour cette issue :)

J'ai fix le test dans le commit c09f975 https://github.com/alelievr/libft-unit-test/commit/c09f975838a45980baa4ebe501756edb31582e77 .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/alelievr/libft-unit-test/issues/43#issuecomment-383291539, or mute the thread https://github.com/notifications/unsubscribe-auth/AeyHJTxNMQ59SAiVIaBm9mjxYMjWwJAuks5tqya9gaJpZM4Tce3x .

alelievr commented 6 years ago

Je viens de fix le bug du test sur strsplit, pour le lstmap tu pourrais expliquer un peu plus le comportement de ta fonction que j'ajuste le test ?