[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH 7/7] error reporting: HACKING: Say to use er
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [RFC PATCH 7/7] error reporting: HACKING: Say to use error_report_errno |
Date: |
Thu, 26 Apr 2018 12:51:24 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 04/26/2018 11:53 AM, Ian Jackson wrote:
> Signed-off-by: Ian Jackson <address@hidden>
> ---
> HACKING | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/HACKING b/HACKING
> index 4125c97..95563a3 100644
> --- a/HACKING
> +++ b/HACKING
> @@ -189,6 +189,7 @@ error_report() or error_vreport() from error-report.h.
> This ensures the
> error is reported in the right place (current monitor or stderr), and in
> a uniform format.
>
> +Use error_report_errno rather than open-coding calls to strerror().
> Use error_printf() & friends to print additional information.
Our style is to write this as error_report_errno(). Or even copy the
approach used in the next line, stating 'error_report_errno() & friends'.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [RFC PATCH 0/7] Introduce error_[v]report_errno[val], Ian Jackson, 2018/04/26
- [Qemu-devel] [RFC PATCH 7/7] error reporting: HACKING: Say to use error_report_errno, Ian Jackson, 2018/04/26
- Re: [Qemu-devel] [RFC PATCH 7/7] error reporting: HACKING: Say to use error_report_errno,
Eric Blake <=
- [Qemu-devel] [RFC PATCH 2/7] error reporting: Provide error_report_errno (and error_vreport_errno), Ian Jackson, 2018/04/26
- [Qemu-devel] [RFC PATCH 5/7] error reporting: Provide error_report_errnoval (and error_vreport_errnoval), Ian Jackson, 2018/04/26
- [Qemu-devel] [RFC PATCH 6/7] error reporting: Use error_report_errnoval in obvious places, Ian Jackson, 2018/04/26