Re: [pve-devel] [PATCH 1/2] Refactored $nokvm to $kvm

2017-08-18 Thread Fabian Grünbichler
On Wed, Aug 16, 2017 at 06:24:29PM +0200, Philip Abernethy wrote: > Where possible I reworked the code to remove the resulting inversion. > The result should be easier to read and understand. small nitpick: commit messages should be written in the present tense / imperative, not past. phrases like

[pve-devel] [PATCH 1/2] Refactored $nokvm to $kvm

2017-08-16 Thread Philip Abernethy
Where possible I reworked the code to remove the resulting inversion. The result should be easier to read and understand. I also fixed a type in the process. --- PVE/QemuServer.pm | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE