ordene por severidade e se guie. se não me engano vc tem uns 9 nivels de "verbosidade" do perlcritic e no mais avantajado (la ele) tem até exemplos. vale a pena.
2012/11/24 Aureliano Guedes <guedes_1...@hotmail.com> > Tem muitas coisas que posso desconsiderar desse perlcritc, ne?! > > agora em > > Regular expression without "/x" flag at line 49, column 18. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#49> See page 236 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html> > > fiquei sem entender, pra que esse maldito /x, não me parece que va mudar > muita coisa. > > Mas legal, se jogar no nivel parece ajudar muito... vou ler o PBP > > Hard tabs used at line 15, column 12. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#15> See page 20 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Hard > tabs used at line 17, column 17. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#17> See page 20 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Close > filehandles as soon as possible after opening them at line 18, column 2. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#18> See page 209 of PBP. > Severity: 4 > <http://perlcritic.com/pod/Perl/Critic/Policy/InputOutput/RequireBriefOpen.html>Ambiguously > named variable "record" at line 20, column 2. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#20> See page 48 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/NamingConventions/ProhibitAmbiguousNames.html>Regular > expression without "/x" flag at line 23, column 18. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#23> See page 236 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Regular > expression without "/x" flag at line 29, column 20. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#29> See page 236 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Hard > tabs used at line 41, column 13. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#41> See page 20 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Hard > tabs used at line 43, column 17. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#43> See page 20 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html>Close > filehandles as soon as possible after opening them at line 44, column 2. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#44> See page 209 of PBP. > Severity: 4 > <http://perlcritic.com/pod/Perl/Critic/Policy/InputOutput/RequireBriefOpen.html>Ambiguously > named variable "record" at line 46, column 2. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#46> See page 48 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/NamingConventions/ProhibitAmbiguousNames.html>Regular > expression without "/x" flag at line 49, column 18. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#49> See page 236 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Regular > expression without "/x" flag at line 56, column 25. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#56> See page 236 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/RegularExpressions/RequireExtendedFormatting.html>Hard > tabs used at line 58, column 4. > <http://perlcritic.com/tmp/ovXTXZfUGR.html#58> See page 20 of PBP. > Severity: 3 > <http://perlcritic.com/pod/Perl/Critic/Policy/CodeLayout/ProhibitHardTabs.html> > > > > ------------------------------ > From: tiago.pecze...@gmail.com > Date: Sat, 24 Nov 2012 18:15:36 -0200 > > To: rio-pm@pm.org > Subject: Re: [Rio-pm] Comparação de arquivos > > use o nivel 3 (mediano) > > 2012/11/24 Aureliano Guedes <guedes_1...@hotmail.com> > > Opa, obrigado então por toda ajuda. > > No mais, so o log do perlcritc. > > http://pastebin.com/esYuR04J > > tenso, ne?! > > > Date: Sat, 24 Nov 2012 18:01:13 -0200 > > > From: br...@rio.pm.org > > To: rio-pm@pm.org > > Subject: Re: [Rio-pm] Comparação de arquivos > > > > 2012/11/24 Aureliano Guedes <guedes_1...@hotmail.com>: > > > Breno, que ser meu orientador... hahaha... > > > > > > Enfim, ficou assim http://pastebin.com/zrpLGwQs > > > > > > Ha algo que você mudaria??? > > > > > > > Sempre há :) > > > > Particularmente, não gosto dessa sua chamada a main() no final do > > arquivo, pq esconde a lógica e dá pouca confiança a quem vai olhar o > > seu código (instruções podem aparecer a qualquer momento fora de uma > > sub, o figura tem que passar por todo o seu script até saber que vc só > > está chamando main()). Eu trocaria a ordem, deixando assim: > > > > --------------------8<-------------------- > > use strict; > > use warnings; > > use autodie; > > > > main(); > > exit; > > > > # declaracoes de sub aqui embaixo > > -------------------->8-------------------- > > > > Assim vc deixa claro o (começo do) fluxo do seu programa e mostra ao > > leitor que não haverão surpresas no meio das subs. > > > > No mais, experimente usar o Perl::Critic (dá até pra usar online via > > perlcritic.com) e veja se ele reclama de alguma coisa. > > > > []s > > > > -b > > _______________________________________________ > > Rio-pm mailing list > > Rio-pm@pm.org > > http://mail.pm.org/mailman/listinfo/rio-pm > > _______________________________________________ > Rio-pm mailing list > Rio-pm@pm.org > http://mail.pm.org/mailman/listinfo/rio-pm > > > > > -- > Tiago B. Peczenyj > Linux User #405772 > > http://pacman.blog.br > > _______________________________________________ Rio-pm mailing list > Rio-pm@pm.org http://mail.pm.org/mailman/listinfo/rio-pm > > _______________________________________________ > Rio-pm mailing list > Rio-pm@pm.org > http://mail.pm.org/mailman/listinfo/rio-pm > -- Tiago B. Peczenyj Linux User #405772 http://pacman.blog.br
_______________________________________________ Rio-pm mailing list Rio-pm@pm.org http://mail.pm.org/mailman/listinfo/rio-pm
