Re: [GSoC][PATCH v2 5/6] virmigraiton: `qemuMigrationJobPhase` transformed for more generic use

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 17, 2020 at 10:37:20AM +0530, Prathamesh Chavan wrote:
> `qemuMigrationJobPhase` was transformed into `virMigrationJobPhase`
> and a common util file `virmigration` was created to store its
> defination.
>
> Signed-off-by: Prathamesh Chavan <pc44800@xxxxxxxxx>
> ---

Like I mentioned in review of v1, moving *just* enums from one module into
another is rarely a desirable change without any additional context. So this
patch should be combined with changes that actually require such a code
movement, either in this series or in a separate one (doesn't matter), but in
its current shape this patch is a little out of place in this series.

Erik




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux