On 11/20/2017 02:38 PM, Fabian Grünbichler wrote:
> On Wed, Nov 15, 2017 at 11:58:29AM +0100, Emmanuel Kasper wrote:
>> Any feedback on this ?
> 
> there already is a "load_defaults" in QemuServer.pm, I'd re-use that.
> what about options with default values besides those four?

I have only those four occurences where defaults where redefined in the
vmstatus() sub.
I sent a new version of the patch using load_defaults()

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

Reply via email to