[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report() |
Date: |
Fri, 07 Jul 2017 14:48:12 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Alistair Francis <address@hidden> writes:
> Convert all uses of error_report("[Ww]arning:"... to use warn_report()
> instead. This helps standardise on a single method of printing warnings
> to the user.
>
> All of the warnings were found using this regex expression:
> error_report.*[Ww]arning:
> and replaced with:
> warn_report("
>
> Signed-off-by: Alistair Francis <address@hidden>
> Suggested-by: Thomas Huth <address@hidden>
Doesn't compile. Moreover, you obviously changed more than just lines
matching the regexp you quoted.
To ease review, I please split the patch as follows:
* First patch:
sed -i 's/error_report("warning: \([^"]*"\)/warn_report("\1/i' ...
plus indentation fixups. Put exact sed command (or whatever else you
use) in the commit message.
* Second patch with additional conversions.
This way the bulk of the changes is mechanical, and the non-mechanical
changes don't get lost in the sea of mechanical ones.
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(), (continued)
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(), Greg Kurz, 2017/07/07
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(), Cornelia Huck, 2017/07/07
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(), Eduardo Habkost, 2017/07/07
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(), Stefan Hajnoczi, 2017/07/07
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(),
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report(), Marcel Apfelbaum, 2017/07/10
[Qemu-devel] [PATCH v1 6/6] Convert error_report*_err() to warn_report*_err(), Alistair Francis, 2017/07/06
[Qemu-devel] [PATCH v1 5/6] error: Implement the warn and free Error functions, Alistair Francis, 2017/07/06
[Qemu-devel] [PATCH v1 4/6] char-socket: Report TCP socket waiting as information, Alistair Francis, 2017/07/06