UCL-INGI / LSINF1252

GNU General Public License v3.0
9 stars 13 forks source link

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

Open eldubuisson opened 5 years ago

eldubuisson commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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.