Skip to content

Commit

Permalink
block/nbd: Use qdict_put()
Browse files Browse the repository at this point in the history
Instead of inlining this nice macro (i.e. resorting to
qdict_put_obj(..., QOBJECT(...))), use it.

Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
  • Loading branch information
XanClic authored and kevmw committed Oct 27, 2016
1 parent 7edca33 commit fcfcd8f
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions block/nbd.c
Expand Up @@ -446,7 +446,7 @@ static void nbd_refresh_filename(BlockDriverState *bs, QDict *options)
QDict *opts = qdict_new();
const char *port = s->port ?: stringify(NBD_DEFAULT_PORT);

qdict_put_obj(opts, "driver", QOBJECT(qstring_from_str("nbd")));
qdict_put(opts, "driver", qstring_from_str("nbd"));

if (s->path && s->export) {
snprintf(bs->exact_filename, sizeof(bs->exact_filename),
Expand All @@ -463,17 +463,16 @@ static void nbd_refresh_filename(BlockDriverState *bs, QDict *options)
}

if (s->path) {
qdict_put_obj(opts, "path", QOBJECT(qstring_from_str(s->path)));
qdict_put(opts, "path", qstring_from_str(s->path));
} else {
qdict_put_obj(opts, "host", QOBJECT(qstring_from_str(s->host)));
qdict_put_obj(opts, "port", QOBJECT(qstring_from_str(port)));
qdict_put(opts, "host", qstring_from_str(s->host));
qdict_put(opts, "port", qstring_from_str(port));
}
if (s->export) {
qdict_put_obj(opts, "export", QOBJECT(qstring_from_str(s->export)));
qdict_put(opts, "export", qstring_from_str(s->export));
}
if (s->tlscredsid) {
qdict_put_obj(opts, "tls-creds",
QOBJECT(qstring_from_str(s->tlscredsid)));
qdict_put(opts, "tls-creds", qstring_from_str(s->tlscredsid));
}

bs->full_open_options = opts;
Expand Down

0 comments on commit fcfcd8f

Please sign in to comment.