> -    my $res = {
> +    my $defaults = {
(...)
> +    foreach my $key (keys %$defaults) {
> +     $res->{$key} = $defaults->{$key} if !$res->{$key};
> +    }

I wonder if this is actually even necessary as parse_config should
simply override old keys.

> +     local $/ = undef;

The `= undef;` part shouldn't be necessary as that's implicit. We seem
to mostly leave it out in our code as far as I can tell.

_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to