On Mon, 14 Mar 2011 15:41:49 -0500
Anthony Liguori<address@hidden> wrote:
On 03/14/2011 03:22 PM, Luiz Capitulino wrote:
On Mon, 14 Mar 2011 14:45:13 -0500
Anthony Liguori<address@hidden> wrote:
On 03/14/2011 02:37 PM, Luiz Capitulino wrote:
qerror_abort() only exists for debugging purposes. I won't say its perfect,
but it's better than nothing and has already saved some time when writing
new errors.
I'm fine dropping it as long as there's a better replacement, which is
not the case here. There's even a hunk that replaces qerror_abort() for
a plain abort().
Yes, that's the replacement.
It's not a good one: it makes the current code a bit worse and we don't know
how and when the error classes are going to be replaced.
Yes, we do, before 0.15.0.
Very optimistic :) I don't doubt you can post patches quickly, but we're
likely going to have fun discussions, respins, tests etc. And all the QAPI
stuff in parallel.