» Snippet : Strip comments of propel classes
By COil on Sunday 4 February 2007 13:02 ::
symfony ::
Permalink ::
Tags ::
optimization
|
propel
|
snippets
|
symfony
|
View comments
[En] In Symfony, comments generated by Propel can sometimes be quiet big. With the 0.6.3 version there was no feature to do this. That's why i made this little batch. We could also imagine extend this snippet to remove all the comments off all php files of an application (for production). If you are using 1.0.0 just add in your "config/propel.ini" (1) :
[Fr] Dans Symfony, les commentaires générés par Propel peuvent parfois être assez gros. Avec la version 0.6.3, il n'y a pas de fonctionnalité pour ne pas les créér. C'est pourquoi j'ai fait ce petit batch. On pourrait aussi imaginer d'étendre ce snippet pour supprimer l'ensembles des commentaires des fichiers php d'une application. (en production). Si vous utilisez la 1.0.0, pas besoin de ce batch cette fonctionnalité est désormais incluse, ajouter dans votre "config/propel.ini" :
(1)
propel.builder.addIncludes = false propel.builder.addComments = false







» Comments
Gagne t-on réellement en perfs en supprimant les commentaires ? Je ne pense pas...
A partir du moment ou on gagne entre 6% et 9% du poids d'un fichier, le parsing sera forcement plus rapide, bien sur sur de petits petits fichiers c'est négligeable, mais sur un fichier qui fait 100k ca commence à faire pas mal... Si Propel propose l'option c'est que ca n'est pas inutile en tout cas, c'est une petite optimisation de plus... rien de transcendant mais c'est toujours ça de gagné.
Dans ce cas, que penser de ce patch refusé que j'avais proposé... www.symfony-project.com/t...
Je ne sais pas, ils considérent le gain comme négligeable...
Mais bon après tout rien ne nous empêche de modifier nos sources comme on le désire et de faire nos modifications.
A la limite s'il n'y a pas d'eaccelerator ou quelque chose dans le genre sur ton serveur tu dois avoir un très léger gain. Mais sur un serveur de prod' par contre, je ne pense pas que ça fasse une différence, surtout en grosse charge, là ou il y aurait besoin.
Par contre en enlevant ces commentaires-là on gagne en lisibilité je trouve.