TP2 - Mc32gest_RS232.c -> bug & amélioration

[PBYetml]

FCT : GetMessage

• => bug => variable => type => normalisé => int8_t - uint8_t - uint16_t => libraire => quelle librairie est associé au type normalisée
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L69C1-L72C56
• => amélioration => cst => define => écriture => convention => comment définissez un define ?
  

  MINF_SLO2_TP/TP2/TP2/firmware/src/Mc32gest_RS232.h

  Line 24 in 10244e3

  #define STX_code (-86) /**< Code de départ (STX), -86 correspond à 0xAA en hex. */

• => amélioration => appel de fct => GetCharFromFifo => paramètre => est-ce que vous ne pourriez pas directement mettre à jour votre structure RxMess à l'appel de la fct ?!
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L94C1-L95C32
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L98C1-L99C32
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L102C1-L103C33
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L106C1-L107C33
• => bug / amélioration / spécification => voir CDC TP1 p3 => algorithme => structure => champs => comment mettez-vous à jour le champs absAngle dans la structure S_pwmSettings
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L125C1-L128C52
• => bug / amélioration / spécification => voir CDC p6 => algorithme => gestion de votre compteur => nbrCycles => pour quoi ne le faire que si la buffer de réception n'est pas à la bonne taille - pourquoi ne pas mettre à jour aussi si la trame est fausse ?!
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L129C1-L153C6

FCT : SendMessage

• => bug => variable => type => normalisé => int8_t - uint16_t => libraire => quelle librairie est associé au type normalisée
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L176C1-L177C51
• => amélioration => cst => define => écriture => convention => comment définissez un define ?
  

  MINF_SLO2_TP/TP2/TP2/firmware/src/Mc32gest_RS232.c

  Line 185 in 10244e3

  ValCRC16Tx = updateCRC16(ValCRC16Tx, STX_code);

• => amélioration => harcodage => écriture => par quoi pourriez-vous remplacer une cts numérique ?
  

  MINF_SLO2_TP/TP2/TP2/firmware/src/Mc32gest_RS232.c

  Line 177 in 10244e3

  uint16_t ValCRC16Tx = 0xFFFF; //Calcul CRC

• => question / documentation => instruction => que fait cette instruction ?
  

  MINF_SLO2_TP/TP2/TP2/firmware/src/Mc32gest_RS232.c

  Line 193 in 10244e3

  PutCharInFifo(&descrFifoTX, (ValCRC16Tx & 0xFF00)>>8); // Place le CRC16 (partie haute) dans le buffer FIFO

FCT : ISR_UART..

• => bug => variable => type => bool => est-ce que ce type fait partie des standards en C ? faut-il lui associé une librairie spécifique ? si oui laquelle ?!
  

  MINF_SLO2_TP/TP2/TP2/firmware/src/Mc32gest_RS232.c

  Line 220 in 10244e3

  bool hwBufferFull;

• => bug => variable => type => normalisé => int8_t - uint8_t => libraire => quelle librairie est associé au type normalisée
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L221C1-L223C26
• => bug / amélioration => algorithme => RTS => MAJ => à quel moment, mettez-vous le signal RTS à 0 dans votre algorithme de l'ISR ?
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L276C1-L280C10
• => amélioration / question => itération => différence entre un while et do..while ?
• => amélioration / question => itération => test / condition => est-ce que faire un test englobant une itération do..while à du sens ?
  https://github.com/etideoliveira/MINF_SLO2_TP/blob/10244e3ec1023d9c0b5a1cf3b30eea3cd4766082/TP2/TP2/firmware/src/Mc32gest_RS232.c#L290C1-L301C1
• [ ]