En même temps le code d'un parseur est généralement bourré de suites de 'while (readline) { if (strncmp(' etc. etc.
Essentiellement dans le fichier que tu incrimines, il n'y a guère que le petit bout pour le "name" qui aurait pu être mis dans une fonction séparée.
Les EINVAL sont une constante clairement définie et clairement retournée pour toutes les erreurs de la grammaire du truc parsé, il n'y a pas de trucs alloués ou incrémentés entre deux tests et retour de fonction hors erreur...
Bref il n'y a pas grand-chose à redire à ce bout de code.
[^] # Re: Ca traduit bien un état d'esprit de la part des développeurs de systemd
Posté par khivapia . En réponse au journal Systemd vs Linux, quand l'intransigeance d'un développeur tourne au ridicule.... Évalué à 2.
En même temps le code d'un parseur est généralement bourré de suites de 'while (readline) { if (strncmp(' etc. etc.
Essentiellement dans le fichier que tu incrimines, il n'y a guère que le petit bout pour le "name" qui aurait pu être mis dans une fonction séparée.
Les EINVAL sont une constante clairement définie et clairement retournée pour toutes les erreurs de la grammaire du truc parsé, il n'y a pas de trucs alloués ou incrémentés entre deux tests et retour de fonction hors erreur...
Bref il n'y a pas grand-chose à redire à ce bout de code.