55Fermer57
squalylLe 05/10/2009 à 10:54
ok, vu.

mais je suis pas d'accord avec ton alloc d'un nombre fixe d'arguments.

32*4+256 octets juste si je veux utiliser 2 args c'est gros. Et si j'en mets plus long que la limite ça se chie dessus silencieusement avec corruption de la heap . hint: snprintf si il est dispo, sinon sprintf dans un buffer temporaire + erreur si ça rentre pas.

il est possible de faire une première passe dans l'estack pour compter les args et leur taille, faire le malloc, et parser ensuite.

j'aurais bien vu la boucle de parsage dans une fonction séparée et un #define DONT_USE_ARGS pour desactiver son utilisation.

et toutes les fonctions "internes" devraient être "static".

désolé j'ai pas le temps de t'envoyer un patch mais ce sont des idées pour ta prochaine version.