diff options
| author | Markus Armbruster <armbru@redhat.com> | 2015-02-06 15:27:19 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2015-02-18 10:50:43 +0100 |
| commit | 2ee2f1e41517efa7b5dd604a68ceca9b1377de2c (patch) | |
| tree | 21d8a1e36e83e83aba5cc549aa1d7a5c2ca06d83 /util | |
| parent | 248337e178fbcf1c20132d4f3d1033cb0dde7638 (diff) | |
error: New convenience function error_report_err()
I've typed error_report("%s", error_get_pretty(ERR)) too many times
already, and I've fixed too many instances of qerror_report_err(ERR)
to error_report("%s", error_get_pretty(ERR)) as well. Capture the
pattern in a convenience function.
Since it's almost invariably followed by error_free(), stuff that into
the convenience function as well.
The next patch will put it to use.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'util')
| -rw-r--r-- | util/error.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/util/error.c b/util/error.c index 2ace0d8dd9..1ff6ae538b 100644 --- a/util/error.c +++ b/util/error.c @@ -152,6 +152,12 @@ const char *error_get_pretty(Error *err) return err->msg; } +void error_report_err(Error *err) +{ + error_report("%s", error_get_pretty(err)); + error_free(err); +} + void error_free(Error *err) { if (err) { |