Changeset
include/qapi/error.h | 8 ++++++++
1 file changed, 8 insertions(+)
Test passed: FreeBSD

loading

Test passed: s390x

loading

Test passed: checkpatch

loading

Test passed: docker

loading

Git apply log
Switched to a new branch '1502359588-29451-1-git-send-email-armbru@redhat.com'
Applying: error: Improve documentation of error_append_hint()
To https://github.com/patchew-project/qemu
 + 69761c85db...2f1144e985 patchew/1502359588-29451-1-git-send-email-armbru@redhat.com -> patchew/1502359588-29451-1-git-send-email-armbru@redhat.com (forced update)
[Qemu-devel] [PATCH] error: Improve documentation of error_append_hint()
Posted by Markus Armbruster, 9 weeks ago
Suggested-by: Halil Pasic <pasic@linux.vnet.ibm.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 include/qapi/error.h | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/include/qapi/error.h b/include/qapi/error.h
index 341b229..c2115a6 100644
--- a/include/qapi/error.h
+++ b/include/qapi/error.h
@@ -230,6 +230,12 @@ void error_prepend(Error **errp, const char *fmt, ...)
 
 /*
  * Append a printf-style human-readable explanation to an existing error.
+ * If the error is later reported to a human user with
+ * error_report_err() or warn_report_err(), the hints will be shown,
+ * too.  If it's reported via QMP, the hints will be ignored.
+ * Intended use is adding helpful hints on the human user interface,
+ * e.g. a list of valid values.  It's not for clarifying a confusing
+ * error message.
  * @errp may be NULL, but not &error_fatal or &error_abort.
  * Trivially the case if you call it only after error_setg() or
  * error_propagate().
@@ -267,11 +273,13 @@ void error_free_or_abort(Error **errp);
 
 /*
  * Convenience function to warn_report() and free @err.
+ * The report includes hints added with error_append_hint().
  */
 void warn_report_err(Error *err);
 
 /*
  * Convenience function to error_report() and free @err.
+ * The report includes hints added with error_append_hint().
  */
 void error_report_err(Error *err);
 
-- 
2.7.5


Re: [Qemu-devel] [PATCH] error: Improve documentation of error_append_hint()
Posted by Eric Blake, 9 weeks ago
On 08/10/2017 05:06 AM, Markus Armbruster wrote:
> Suggested-by: Halil Pasic <pasic@linux.vnet.ibm.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  include/qapi/error.h | 8 ++++++++
>  1 file changed, 8 insertions(+)

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PATCH] error: Improve documentation of error_append_hint()
Posted by Halil Pasic, 9 weeks ago

On 08/10/2017 12:06 PM, Markus Armbruster wrote:
> Suggested-by: Halil Pasic <pasic@linux.vnet.ibm.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Nice clarification!

Reviewed-by: Halil Pasic <pasic@linux.vnet.ibm.com>