[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v2 1/4] block: Allow changing bs->file on reopen
From: |
Alberto Garcia |
Subject: |
Re: [RFC PATCH v2 1/4] block: Allow changing bs->file on reopen |
Date: |
Tue, 16 Feb 2021 13:06:00 +0100 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 10 Feb 2021 05:52:47 PM CET, Kevin Wolf wrote:
>> + /* The 'file' option only allows strings */
>> + assert(qobject_type(value) == QTYPE_QSTRING);
>
> This is true, but not entirely obvious: The QAPI schema has
> BlockdevRef, which can be either a string or a dict. However, we're
> dealing with a flattened options dict here, so no more nested dicts.
You're right, I'll update the comment.
Berto