|
From: | Anthony Liguori |
Subject: | [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error |
Date: | Fri, 16 Oct 2009 12:47:44 -0500 |
User-agent: | Thunderbird 2.0.0.23 (X11/20090825) |
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
[Prev in Thread] | Current Thread | [Next in Thread] |