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

Reading from the wire - Pas besoin de l'appel à modem_read #23

Open
eldubuisson opened this issue Mar 19, 2019 · 2 comments
Open

Reading from the wire - Pas besoin de l'appel à modem_read #23

eldubuisson opened this issue Mar 19, 2019 · 2 comments

Comments

@eldubuisson
Copy link

Pour réussir tous les tests de reading from the wire [S3] il n'est pas nécessaire d'appeler la fonction mode_read, le buffer mis en argument est déjà rempli avec le bon tableau de char.

@Maxmawt
Copy link
Collaborator

Maxmawt commented Mar 21, 2019

Si, il est quand-même nécessaire d'appeler modem_read() étant donné que la fonction renvoie aussi le nombre de bytes écrits. Sans ce nombre, tu risques d'avoir une segfault.

@KilianMaes
Copy link

Je confirme qu'il faudrait améliorer le feedback donné par Inginious pour cet exercice. Personnellement, je pensais que le buffer donné en argument était celui résultant d'un appel à modem-read(), et que le fonction modem_read() était donnée à titre indicatif. Le code suivant me permettait d'obtenir la note de 83,33%.
int i;
for(i = 0; i < 256; i++)
if(buf[i] == 42) return 1;
return 0;
Cela peut porter à confusion, car l'étudiant pense que l'utilisation de modem_read() n'est pas nécessaire. Il serait donc utile soit de vérifier que la solution proposée fait bien appel à modem_read(), soit de clarifier l'énoncé en précisant qu'il faut "remplir" le buffer à l'aide de modem_read() avant de l'utiliser.

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

3 participants