[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH 14/14] block: Stop passing flags to bdrv_reopen_queue_child() |
Date: |
Tue, 09 Oct 2018 10:44:44 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Mon 08 Oct 2018 08:46:57 PM CEST, Max Reitz wrote:
> On 08.10.18 20:13, Alberto Garcia wrote:
>> On Mon 08 Oct 2018 04:48:50 AM CEST, Max Reitz wrote:
>>
>>>> + /* Old values are used for options that aren't set yet */
>>>> + old_options = qdict_clone_shallow(bs->options);
>>>> + bdrv_join_options(bs, options, old_options);
>>>> + qobject_unref(old_options);
>>>> +
>>>> + /* We have the final set of options so let's update the flags */
>>>> + {
>>>> + Error *local_err = NULL;
>>>> + QemuOpts *opts;
>>>> + QDict *options_copy = qdict_clone_shallow(options);
>>>
>>> I-I'm not sure this conforms to our coding style.
>>>
>>> While I applaud your effort to keep the patch size small, I know for
>>> sure I don't want to read code like this. (Unless it's necessary
>>> because of some variables' lifetimes.)
>>
>> I actually think it makes the code more readable if there are variables
>> with a very limited scope (like in this case). But I can also rewrite it
>> using a more traditional style.
>
> I think it should be a function call then.
Nah, I don't think this deserves its own function. I'd rather keep
everything in the same function. I'll send an updated patch.
Berto