Salut,

J'ai peut être trouvé quelque chose de mon modeste niveau pour aider au code de dotclear. J'ai ouvert un premier ticket et fait une Pull Request (https://bitbucket.org/dotclear/dotclear/pull-request/38/ticket-1461-replace-all-calls-to/diff) en espérant avoir tout bien fait comme il faut :)

Je regardais un peu les fonctions de clearbricks et j'ai trouvé cette méthode qui réinvente la roue puisqu'il existe une même fonction native (et pas récente, je vous parle de PHP 4 là ;) ) qui fait exactement la même chose.

J'ai ensuite regardé une autre méthode ( path:real() ), et ça fait exactement la même chose que la fonction realpath() native de php depuis 5.0. Au passage j'ai fait un benchmark et la méthode clearbricks est 20 fois plus lente que la méthode native.

Juste dans lib.files.php, je vois plusieurs autres méthodes qui me semblent bien exister en PHP natif, même si on se cantonne à 5.2

Avant que je fasse d'autres pull requests pour remplacer ces méthodes qui réinventent la roue, sont probablement moins fiables que les fonctions natives et sont plus lentes, est-ce qu'il y a une bonne raison qui m'a échappée pour qu'elles aient été créées (en dehors de la dette technique bien sûr) ?

Merci

Pascal
--
Dev mailing list - [email protected] - http://ml.dotclear.org/listinfo/dev

Répondre à