2012/7/20 Guillaume Rousse <[email protected]>:
> Le 19/07/2012 11:46, Gonéri Le Bouder a écrit :
> Excellent idea, but I'd prefer to keep multi valued option handling in
> Config.pm, and deal with perl lists in the rest of the code.
It deserve
>
> Also, minor cosmetic issues:
> - every non-trivial loop should use explicit iterators:
> foreach (split(/,/, $self->{trust})) -> foreach my $something (split(/,/,
> $self->{trust}))
> - error message format is usually: "can't do something: $error", not "can't
> do something - $error"
Hi Guillaume,
As you may have notified, I commited a patch with your comment.
Regards,
--
Gonéri Le Bouder
_______________________________________________
Fusioninventory-devel mailing list
[email protected]
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/fusioninventory-devel
