aboutsummaryrefslogtreecommitdiff
path: root/include/qemu/option.h
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2015-02-12 17:07:34 +0100
committerMarkus Armbruster <armbru@redhat.com>2015-02-26 14:47:49 +0100
commit79087c782e1549a6f9c8303aafc0b74f4e637756 (patch)
tree32261ec7ed405e6118a7e618b0e9f0cf8d248e3a /include/qemu/option.h
parent39101f2511f6adf1ae4380f8d729dba1213b9d7a (diff)
downloadqemu-79087c782e1549a6f9c8303aafc0b74f4e637756.zip
qemu-79087c782e1549a6f9c8303aafc0b74f4e637756.tar.gz
qemu-79087c782e1549a6f9c8303aafc0b74f4e637756.tar.bz2
QemuOpts: Convert qemu_opts_set() to Error, fix its use
Return the Error object instead of reporting it with qerror_report_err(). Change callers that assume the function can't fail to pass &error_abort, so that should the assumption ever break, it'll break noisily. Turns out all callers outside its unit test assume that. We could drop the Error ** argument, but that would make the interface less regular, so don't. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'include/qemu/option.h')
-rw-r--r--include/qemu/option.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/include/qemu/option.h b/include/qemu/option.h
index 7422cc2..7d6addc 100644
--- a/include/qemu/option.h
+++ b/include/qemu/option.h
@@ -110,8 +110,8 @@ QemuOpts *qemu_opts_create(QemuOptsList *list, const char *id,
int fail_if_exists, Error **errp);
void qemu_opts_reset(QemuOptsList *list);
void qemu_opts_loc_restore(QemuOpts *opts);
-int qemu_opts_set(QemuOptsList *list, const char *id,
- const char *name, const char *value);
+void qemu_opts_set(QemuOptsList *list, const char *id,
+ const char *name, const char *value, Error **errp);
const char *qemu_opts_id(QemuOpts *opts);
void qemu_opts_set_id(QemuOpts *opts, char *id);
void qemu_opts_del(QemuOpts *opts);