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