Posté par errno .
En réponse au message Terminé.
Évalué à 8.
Je n'ai pas le temps de regarder en profondeur mais au premier coup d'oeil, j'ai regardé le fichier mm_core_parser.c, et j'ai remarqué quelques points :
pourquoi rajouter les prototypes au début du .c alors qu'ils sont dans le .h ?
dans ParseAlloc(), tu fais :
parse_t *pRet = NULL;
pRet = malloc(sizeof(parse_t));
plutot que de mettre sizeof(parse_t), tu devrais mettre sizeof(*pRet) pour qu'un changement de type n'ai pas a etre modifié partout.
de plus, pourquoi ne pas faire directement :
parse_t *pRet = malloc(sizeof(*pRet));
à plusieurs endroits tu utilises l'expression a ? b : c de cette facon :
( cond ) ? a = b : a = c;
c'est mieux de faire :
a = ( cond ) ? b : c;
Ok je cherche la petit bête ;).
A part ça le code est bien commenté et se lit bien et je ne pourrais que t'encourager à le mettre en ligne, cela peut toujours interesser des gens, ne serait ce qu'a but éducatif.
Bonne continuation
# question de style
Posté par errno . En réponse au message Terminé. Évalué à 8.
parse_t *pRet = malloc(sizeof(*pRet));( cond ) ? a = b : a = c;c'est mieux de faire :a = ( cond ) ? b : c;Ok je cherche la petit bête ;). A part ça le code est bien commenté et se lit bien et je ne pourrais que t'encourager à le mettre en ligne, cela peut toujours interesser des gens, ne serait ce qu'a but éducatif. Bonne continuation