On Mon, Oct 02, 2017 at 10:07:01AM +0200, Andrea Bolognani wrote:
On Fri, 2017-09-29 at 16:49 +0200, Martin Kletzander wrote:
> + <description>
> + While waiting for a write to disk to be finished, e.g. during save,
> + even simple operations like <code>virsh list</code> would be
blocking
> + due to domain lock. This is now resolved by unlocking the domain
> + in places where it is not needed.
s/ / /g
here and further below. I know you prefer double-space-after-period,
but we are consistently using a single space in this file :)
Sure, it is a muscle memory at this point. Anyway:
:%s_\. \([A-Z]\)_. \1_gc
showed one already existing line that I fixed as a part of this.
> + <description>
> + Previously the documentation used multiples of 1000, but now it is
> + fixe to use multiples of 1024.
s/fixe/fixed/
> + <change>
> + <summary>
> + Fix message decoding which was caused very strange bug
s/caused/causing/
s/bug/bugs/
I believe it was a bug, so I added the indefinite article instead of this last change.
With these issues fixed,
Reviewed-by: Andrea Bolognani <abologna(a)redhat.com>
Thanks for once again picking up the slack :)
Sure, you could also cut me some slack with the double spaces since this is
fixed width style file ;)
Pushed now, thanks.
--
Andrea Bolognani / Red Hat / Virtualization