Re: [Libconf-dev] libconf/perl-Libconf Libconf.pm Glueconf/Hosts....

[dams]

Thierry Vignaud <address@hidden> said:

> CVSROOT:      /cvsroot/libconf
> Module name:  libconf
> Branch:       
> Changes by:   Thierry Vignaud <address@hidden>        03/11/27 05:23:47
>
> Modified files:
>       perl-Libconf   : Libconf.pm 
>       perl-Libconf/Glueconf: Hosts.pm KeyValue.pm Passwd.pm Resolv.pm 
>                              Samba.pm Shell.pm Ups.pm ValuesSection.pm 
>                              XF86Config.pm 
>       perl-Libconf/System: DirectoryWrapper.pm 
>       perl-Libconf/System/MDK: Users.pm 
>
> Log message:
>       make perl_checker happier

C'est pas un message de log satisfaisant. Répond à ce mail en disant ce que tu
as modifié, merci, même si c'est long et chiant.


 # COMMENTS TO BE REVIEWED
@@ -98,8 +98,7 @@
 #   -2 if the template didn't succeed finding the mathing atom ($index was
 -1),
 #   -3 if there is an error setting the values in the atom
 sub editAtom {
-    my ($out, $index, $ref_args) = @_;
-    my %args=%$ref_args;


 ^^^^^
Ca casse tout, le %args est necessaire, il est utilisé dans le eval.
Tu commence à m'echauffer serieusement les oreilles.

Pareil pour tous les autres endroits.

Changes tout ça.

ET NE COMMITES PLUS SANS FAIRE make test. Ya des regles pour ça, j'aurais du te 
dire



-- 
dams