[Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error

[Anthony Liguori]

Paolo Bonzini wrote:
> On 10/16/2009 04:17 PM, Luiz Capitulino wrote:
> >   Second, I fear we are going too far with the objects idea. It
> > solves the monitor's problem well and although can be used by
> > other subsystems I wonder if pushing them to the extreme like that
> > is the way to go.
>
> I think that in practice Anthony is asking you only to write the 
> stringified enum name (e.g. "QEMU_ERROR_NODEV"), instead of the enum 
> value (i.e. 2). :-)

Yup.

> I agree about what the immediate needs are.  On the other hand, making 
> things too complicated to extend later is not a good idea...

There are just a few things we have to get right and we should take our 
time on those.  The error semantics are one of those important things 
that we'll regret in the future if we rush.

--
Regards,

Anthony Liguori