[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 01/28] qemu-img: stop printing error twice in a few places
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 01/28] qemu-img: stop printing error twice in a few places |
Date: |
Mon, 26 Feb 2024 14:14:39 +0000 |
User-agent: |
Mutt/2.2.12 (2023-09-09) |
On Thu, Feb 22, 2024 at 12:15:42AM +0300, Michael Tokarev wrote:
> Currently we have:
>
> ./qemu-img resize none +10
> qemu-img: Could not open 'none': Could not open 'none': No such file or
> directory
>
> stop printing the message twice, - local_err already has
> all the info, no need to prepend additional text there.
>
> There are a few other places like this, but I'm unsure
> about these.
>
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
> qemu-img.c | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [PATCH v2 00/28] qemu-img: refersh options and --help handling, cleanups, Michael Tokarev, 2024/02/21
- [PATCH 01/28] qemu-img: stop printing error twice in a few places, Michael Tokarev, 2024/02/21
- [PATCH 08/28] qemu-img: check: refresh options/--help, Michael Tokarev, 2024/02/21
- [PATCH 05/28] qemu-img: pass current cmd info into command handlers, Michael Tokarev, 2024/02/21
- [PATCH 02/28] qemu-img: measure: convert img_size to signed, simplify handling, Michael Tokarev, 2024/02/21
- [PATCH 03/28] qemu-img: create: convert img_size to signed, simplify handling, Michael Tokarev, 2024/02/21
- [PATCH 04/28] qemu-img: global option processing and error printing, Michael Tokarev, 2024/02/21