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

[Thierry Vignaud]

address@hidden writes:

> > 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.

- remplacement des appels a "->methode()" par "->methode"
- virer les espaces apres "(" et avant ")"
- ajouter des espaces apres "{" et avant "}" (eg entre ")" et "{")
- marquer les variables non utilisees en les prefixant avec "_"
- ajout de () aux appels de fonctions sans parametres comme wantarray
  ou debug pour que perl_checker ne les confonde pas avec des
  variables non declarees

>  # 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.

elle n'est pas utilisee dans le eval qui suit.

eg dans Libconf.pm, il n'y avait qu'un seul %args reellement utilise