On Thu, Oct 01, 2015 at 03:13:31PM +0200, Kevin Wolf wrote:
> +static void change_parent_backing_link(BlockDriverState *from,
> +                                       BlockDriverState *to)
> +{
> +    BdrvChild *c, *next;
> +
> +    QLIST_FOREACH_SAFE(c, &from->parents, next_parent, next) {

I find the name "c" confusing because it reminds me of children rather
than the parents list.

Perhaps "parent" would be clearer here.

Reply via email to