On Mon, Sep 01, 2014 at 05:22:04PM +0800, Fam Zheng wrote:
> +{ 'type': 'BlockdevOptionsNull',
> +  'base': 'BlockdevOptionsFile',
> +  'data': { '*size': 'int' } }

Why inherit from BlockdevOptionsFile?

No other blockdev options type does that.  Unless there is a real
inheritance relationship, please don't do this.

Stefan

Attachment: pgpFV8gxYR8BS.pgp
Description: PGP signature

Reply via email to