<div dir="ltr">Cool et merci de nous tenir au courant !<div> </div><div>J'ai pas tout suivi mais apparemment vous vous êtes basés sur cet article par Ovid :</div><div> </div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>I know that <code style="margin:0px;padding:2px;border:0px;font-size:14px;vertical-align:baseline;color:rgb(51,51,51)">Veure::Module</code> won't meet your exact needs, but the idea of building standards into a single module is worth the trouble. Combine that with <code style="margin:0px;padding:2px;border:0px;font-size:14px;vertical-align:baseline;color:rgb(51,51,51)">Perl::Critic</code> and <code style="margin:0px;padding:2px;border:0px;font-size:14px;vertical-align:baseline;color:rgb(51,51,51)">Perl::Tidy</code> and you're well on your way to reining in the headache of large codebases.</div></blockquote><div><div> </div><div><a href="http://blogs.perl.org/users/ovid/2019/03/enforcing-simple-standards-with-one-module.html" target="_blank">http://blogs.perl.org/users/ovid/2019/03/enforcing-simple-standards-with-one-module.html</a> </div></div><div> </div><div>k</div></div> <div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 6 Jul 2022 at 13:51, Stéphane Gigandet <<a href="mailto:biz@joueb.com" target="_blank">biz@joueb.com</a>> wrote: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello, Suite à ce message, Sébastien Feugère (un grand merci à lui) a répondu en commentaire dans l'issue sur GitHub, et il nous a pointé vers Veure::Module. On a donc activé les signatures, avec un module pour remplacer le Modern::Perl qu'on utilisait jusqu'alors pour le "boilerplate" : <a href="https://github.com/openfoodfacts/openfoodfacts-server/pull/7009" rel="noreferrer" target="_blank">https://github.com/openfoodfacts/openfoodfacts-server/pull/7009</a> package ProductOpener::PerlStandards; use 5.24.0; use strict; use warnings; use feature (); use utf8; sub import { warnings->import; warnings->unimport('experimental::signatures'); strict->import; feature->import(qw/signatures :5.24/); utf8->import; } sub unimport { warnings->unimport; strict->unimport; feature->unimport; utf8->unimport; } 1; Bonne après midi, Stéphane Le 30/06/2022 à 10:08, Stéphane Gigandet a écrit : > Bonjour, > > Dans le code d'Open Food Facts, on a pris l'habitude d'utiliser les > prototypes des fonctions pour indiquer le nombre de paramètres attendus. > > Du genre : > > sub ma_fonction($$) { > > my $param1 = shift; > > my $param2 = shift; > > .. > > } > > C'est pratique car si j'appelle ma_fonction("un seul paramètre"), j'ai > une erreur à la compilation. > > Mais il y a plein de posts qui expliquent quel les prototypes en Perl > c'est mal et qu'il ne faut pas les utiliser comme ça. En pratique > aucune de nos fonctions n'est appelée avec des listes en paramètres, > seulement des scalar / des ref. > > On se pose la question de changer ces prototypes : les garder, les > enlever, les remplacer par les signatures > <a href="https://perldoc.pl/perlsub#Signatures" rel="noreferrer" target="_blank">https://perldoc.pl/perlsub#Signatures</a> ? > > Le début de discussion est là : > <a href="https://github.com/openfoodfacts/openfoodfacts-server/issues/6956" rel="noreferrer" target="_blank">https://github.com/openfoodfacts/openfoodfacts-server/issues/6956</a> > > Je me demandais ce que vous faisiez dans vos projets ? > > Merci et bonne journée (sous la pluie ?), > > Stéphane > > _______________________________________________ > Perl mailing list > <a href="mailto:Perl@mongueurs.net" target="_blank">Perl@mongueurs.net</a> > <a href="http://listes.mongueurs.net/mailman/listinfo/perl" rel="noreferrer" target="_blank">http://listes.mongueurs.net/mailman/listinfo/perl</a> > Attention, les archives sont publiques _______________________________________________ Perl mailing list <a href="mailto:Perl@mongueurs.net" target="_blank">Perl@mongueurs.net</a> <a href="http://listes.mongueurs.net/mailman/listinfo/perl" rel="noreferrer" target="_blank">http://listes.mongueurs.net/mailman/listinfo/perl</a> Attention, les archives sont publiques </blockquote></div>