Re: [Qemu-devel] [PATCH v5 07/11] blockdev: rename BlkTransactionStates to singular

[Kevin Wolf]

Am 30.05.2013 um 14:34 hat Stefan Hajnoczi geschrieben:
> The QMP 'transaction' command keeps a list of in-flight transactions.
> The transaction state structure is called BlkTransactionStates even
> though it only deals with a single transaction.  The only plural thing
> is the linked list of transaction states.
> 
> I find it confusing to call the single structure "States".  This patch
> renames it to "State", just like BlockDriverState is singular.
> 
> Signed-off-by: Stefan Hajnoczi <address@hidden>

> -    /* This removes our old bs from the bdrv_states, and adds the new bs */
> -    bdrv_append(states->new_bs, states->old_bs);
> +    /* This removes our old bs and adds the new bs */
> +    bdrv_append(state->new_bs, state->old_bs);

Tsk, you said this was only renaming. Don't believe I didn't see it!

Reviewed-by: Kevin Wolf <address@hidden>