On 11/20/2014 10:06 AM, Max Reitz wrote: > Progress may regress; this should be displayed correctly by > qemu_progress_print(). > > Signed-off-by: Max Reitz <mre...@redhat.com> > --- > util/qemu-progress.c | 1 + > 1 file changed, 1 insertion(+)
Reviewed-by: Eric Blake <ebl...@redhat.com> > > diff --git a/util/qemu-progress.c b/util/qemu-progress.c > index 4ee5cd0..c0fb14d 100644 > --- a/util/qemu-progress.c > +++ b/util/qemu-progress.c > @@ -152,6 +152,7 @@ void qemu_progress_print(float delta, int max) > state.current = current; > > if (current > (state.last_print + state.min_skip) || > + current < (state.last_print - state.min_skip) || > (current == 100) || (current == 0)) { Maybe we can drop the redundant () as long as we are touching this area of code? But not worth a respin. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature