[pve-devel] [PATCH] fix raise_param_exc in PVE::QemuServer::Memory::memory_unplug
Dietmar Maurer
dietmar at proxmox.com
Wed Jun 8 18:55:33 CEST 2016
> >>Sigh, and we also have a cyclic include:
> >>
> >>PVE::QemuServer => PVE::QemuServer::Memory => PVE::QemuServer
>
> Is it really a problem ? (you are better than me with Perl)
No, it is no 'real' problem. But it is not nice, and I think
we should try to avoid it.
> >>and ideas how to avoid that?
>
> Here the list of sub from QemuServer.pm used in QemuServer::Memory
>
> PVE::QemuServer::vm_mon_cmd
> PVE::QemuServer::qemu_objectdel
> PVE::QemuServer::qemu_devicedel
> PVE::QemuServer::vm_mon_cmd_nocheck
>
> -> could be move in a new QemuServer::Commands for example
>
> PVE::QemuServer::parse_hotplug_features
>
> -> could be move in a new QemuServer::Config for example
Yes, something like that.
More information about the pve-devel
mailing list