On Mon, Jul 23, 2012 at 9:57 PM, Laszlo Ersek <[email protected]> wrote: > On 07/20/12 14:01, Stefan Hajnoczi wrote: >> @@ -970,7 +895,7 @@ void qmp_set_link(const char *name, bool up, Error >> **errp) >> done: >> >> if (!vc) { >> - error_set(errp, QERR_DEVICE_NOT_FOUND, name); >> + qerror_report(QERR_DEVICE_NOT_FOUND, name); >> return; >> } > > Why? The only caller (at this point) seems to be hmp_set_link(), and it > handles the error (with monitor_printf()).
Looks like a rebase/merge issue. Thanks for pointing it out. Stefan
