Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

42 file checker, mauvais test pour ft_strcmp #33

Open
Gamouche opened this issue Nov 11, 2017 · 1 comment
Open

42 file checker, mauvais test pour ft_strcmp #33

Gamouche opened this issue Nov 11, 2017 · 1 comment

Comments

@Gamouche
Copy link

Bonjour/soir,

Lorsque 42 file checker via Maintest teste la fonction ft_strcmp(), vous vous attendez à ce que la fonction renvoie la difference entre les 2 caractères qui diffèrent. Tester la fonction de cette manière est une très mauvaise idée puisque strcmp() renvoie un entier positif, nul ou négatif ... pas forcément la difference entre les 2 caractères qui diffèrent. Si s1 est inférieur à s2, renvoyer -1 est tout aussi valide que de renvoyer -666. Il vous faut donc non pas regarder si notre ft_strcmp() renvoie la même valeur que strcmp(), mais regarder si la valeur est bien négative, positive ou nulle.
cyfermie

@QuentinPerez
Copy link
Owner

Salut,
Merci pour le report mais c'est les builtins du compilo qui ont change entre temps.
Si tu es sur que tous les macs de l'ecole sont a jour tu peux me faire une PR pour fixer ca :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants