[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/18] qerror: introduce QERR_GENERIC_ERROR
From: |
Luiz Capitulino |
Subject: |
Re: [Qemu-devel] [PATCH 01/18] qerror: introduce QERR_GENERIC_ERROR |
Date: |
Thu, 30 Aug 2012 09:11:43 -0300 |
On Wed, 15 Aug 2012 09:41:42 +0200
Pavel Hrdina <address@hidden> wrote:
> Signed-off-by: Pavel Hrdina <address@hidden>
> ---
> qerror.h | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/qerror.h b/qerror.h
> index d0a76a4..7e0bae7 100644
> --- a/qerror.h
> +++ b/qerror.h
> @@ -120,6 +120,9 @@ void assert_no_error(Error *err);
> #define QERR_FEATURE_DISABLED \
> ERROR_CLASS_GENERIC_ERROR, "The feature '%s' is not enabled"
>
> +#define QERR_GENERIC_ERROR \
> + ERROR_CLASS_GENERIC_ERROR, "An (Errno %d) error has occurred"
> +
You should use error_setg() instead:
http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg04980.html
There usage examples in the series introducing it. It would be better
to wait for it to be merged before you use it though, as it's always
possible for people to ask for changes.
> #define QERR_INVALID_BLOCK_FORMAT \
> ERROR_CLASS_GENERIC_ERROR, "Invalid block format '%s'"
>
[Qemu-devel] [PATCH 05/18] block: add error parameter to bdrv_snapshot_list() and related functions, Pavel Hrdina, 2012/08/15
[Qemu-devel] [PATCH 04/18] block: add error parameter to bdrv_snapshot_delete() and related functions, Pavel Hrdina, 2012/08/15
[Qemu-devel] [PATCH 06/18] block: add error parameter to bdrv_snapshot_find(), Pavel Hrdina, 2012/08/15
[Qemu-devel] [PATCH 07/18] block: add error parameter to del_existing_snapshots(), Pavel Hrdina, 2012/08/15