Re: [Qemu-devel] [PATCH v2 1/1] s390x: vmstatify config migration for virtio-ccw

[Eric Blake <eblake@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1114 bytes --]

On 06/04/2017 10:09 PM, Dong Jia Shi wrote:
> * Halil Pasic <pasic@linux.vnet.ibm.com> [2017-06-02 16:05:31 +0200]:
> 
> Hi Halil,
> 
> Sorry for the late show up. I just found some nits, which could be
> ignored for me.
> 
>> Let's vmstatify virtio_ccw_save_config and virtio_ccw_load_config for
>> flexibility (extending using subsections) and for fun.
>>
>> To achieve this we need to hack the config_vector, which is VirtIODevice
>> (that is common virtio) state, in the middle of the VirtioCcwDevice state
>> representation.  This is somewhat ugly, but we have no choice because the
>                  ^^
> Nit:-------------++

What's wrong here?  Two spaces between sentences is a common
typographical convention (true, the codebase is inconsistent on whether
sentences are separated with one or two spaces, but that's all the more
reason to realize that since we don't have a consistent standard, it is
just churn to change from one style to the other)

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]