Mc32gest_RS232.c -> remarques / améliorations / bugs

[PBYetml]

FCT : GetMessage

• [ ] => bug / amélioration / question => types => librairie => norme en C => quelles sont les conventions d'écriture par rapport à la gestion des librairies ?
• [ ] => bug / amélioration / question => types => uint8_t - uint16_t => librairie => quelle librairie doit être associé à ces types ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L96C1-L102C5
• [ ] => bug / amélioration / question => algorithme => récupération CRC => pourquoi ne pas utiliser la notion de l'union ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L124C1-L126C60
• [ ] => bug / amélioration / question => algorithme => nombre de cycle => que ce passe-t-il si il y a plus de cycles faux que la valeur max ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L143C1-L159C6

FCT : SendMessage

• [ ] => bug / amélioration / question => types => uint8_t - uint16_t => librairie => quelle librairie doit être associé à ces types ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L189C1-L190C70
• [ ] => amélioration / question => constante VS define => Hardcodage => quelle différence entre 0xAA et START_BYTE ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L198C46-L198C56 https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L203
• [ ] => bug / amélioration / question => algorithme => comment gérez-vous la taille disponible dans le buffer d'envoi => freeSize => après l'écriture des paramètres ? n'y a-t-il pas une instruction manquante ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L211

FCT : ISR => UART

• [ ] => bug / amélioration / question => types => uint8_t - int8_t - uint16_t => librairie => quelle librairie doit être associé à ces types ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L226C1-L231C22
• [ ] => bug / amélioration / question => type => BOOL => d'ou vient ce type => à quelle librairie faites-vous référence ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L232
• [ ] => amélioration / question => algorithme => 'interruption de l'ISR (UART) => partie réception => à quel moment est activée cette interruption ? est-ce que le buffer hardware est forcément vide après la lecture de la data ?
• [ ] => bug / amélioration / question => algorithme => comment déterminez-vous si la FIFO software est pleine ou non ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L268C1-L274C22
• [ ] => amélioration / question => test / condition => algorithme => constante numérique - hardcodage => pourquoi la valeur de 6 ? but du test ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L291C1-L292C28
• [ ] => question / amélioration => variable intermédiaire => à quoi vous sert la variable i_cts ? ne pouvez-vous pas directement utiliser RS232_CTS ? d’où vient cette constante ? et que représente-t-elle ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L314 https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L322C1-L322C36

AUTRE

• [ ] => amélioration / question => define => déclaration => convention C => écriture => où devriez-vous déclarer vos différents define ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L32C1-L34C21
• [ ] => bug / amélioration / question => define => compréhension code existant ? quelle différence ? https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L30 https://github.com/antoniodocarmoafonso/TP2-MINF/blob/a31ff98199ebb0b6823d29563065267a52b3219e/Mc32gest_RS232.c#L32
• [ ]

[PBYetml]

L'évaluation complète sur le fichier : Mc32gest_RS232.c est fini => vous pouvez si vous les désirer faire réaliser les amélioration ou correction sur ce fichier