On 10/20/2017 10:11 AM, Kevin Wolf wrote: > Am 12.10.2017 um 15:20 hat Eric Blake geschrieben: >> On 10/11/2017 10:47 PM, Eric Blake wrote: >>> Now that bdrv_is_allocated accepts non-aligned inputs, we can >>> remove the TODO added in earlier refactoring. >>> >>> Signed-off-by: Eric Blake <ebl...@redhat.com> >> >> I was a bit too hasty in the copy-and-paste from 22/24 - there is no >> assertion in this patch, so the subject would be better as: >> >> qcow2: Reduce is_zero() rounding > > In fact, I don't see any assertion being modified in patch 22 either.
Oh, indeed. Not sure what I was thinking; I guess it's because I added 22 at the same time I added several other TODO assertions to series 1 a couple months ago, with the plans to relax rounding/assertions as appropriate later in the series, and then got confused on which things were being relaxed where ;) If it's just the subject lines for 22 and 23 that need fixing, I trust you can do that as maintainer. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature