|
From: | Paolo Bonzini |
Subject: | [Qemu-devel] Re: [PATCH 6/9] QError: Add qdev not found error |
Date: | Fri, 16 Oct 2009 19:28:47 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20091014 Fedora/3.0-2.8.b4.fc11 Lightning/1.0pre Thunderbird/3.0b4 |
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). :-)
I agree about what the immediate needs are. On the other hand, making things too complicated to extend later is not a good idea...
Paolo, who's amazed at the amount of design changes that went into QEMU since last June...
[Prev in Thread] | Current Thread | [Next in Thread] |