[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io wi
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o |
Date: |
Thu, 18 Mar 2010 19:58:00 +0200 |
On 3/17/10, Markus Armbruster <address@hidden> wrote:
> Blue Swirl <address@hidden> writes:
>
> > On 3/17/10, Markus Armbruster <address@hidden> wrote:
> >> The location tracking interface is used by code shared with qemi-img,
> >> qemu-nbd and qemu-io, so it needs to be available there. Commit
> >> 827b0813 provides it in a rather hamfisted way: it adds a dummy
> >> implementation to qemu-tool.c.
> >>
> >> It's cleaner to provide the real thing, and put a few more dummy
> >> monitor functions into qemu-tool.c.
> >>
> >> Signed-off-by: Markus Armbruster <address@hidden>
> >> ---
> >> Makefile | 6 +++---
> >> qemu-tool.c | 50 ++++++++++++++++----------------------------------
> >> 2 files changed, 19 insertions(+), 37 deletions(-)
> >>
> >> diff --git a/Makefile b/Makefile
> >> index 2066c12..aad361e 100644
> >> --- a/Makefile
> >> +++ b/Makefile
> >> @@ -129,11 +129,11 @@ bt-host.o: QEMU_CFLAGS += $(BLUEZ_CFLAGS)
> >> qemu-img.o: qemu-img-cmds.h
> >> qemu-img.o qemu-tool.o qemu-nbd.o qemu-io.o: $(GENERATED_HEADERS)
> >>
> >> -qemu-img$(EXESUF): qemu-img.o qemu-tool.o $(block-obj-y)
> $(qobject-obj-y)
> >> +qemu-img$(EXESUF): qemu-img.o qemu-tool.o qemu-error.o $(block-obj-y)
> $(qobject-obj-y)
> >>
> >> -qemu-nbd$(EXESUF): qemu-nbd.o qemu-tool.o $(block-obj-y)
> $(qobject-obj-y)
> >> +qemu-nbd$(EXESUF): qemu-nbd.o qemu-tool.o qemu-error.o $(block-obj-y)
> $(qobject-obj-y)
> >>
> >> -qemu-io$(EXESUF): qemu-io.o qemu-tool.o cmd.o $(block-obj-y)
> $(qobject-obj-y)
> >> +qemu-io$(EXESUF): qemu-io.o cmd.o qemu-tool.o qemu-error.o
> $(block-obj-y) $(qobject-obj-y)
> >>
> >> qemu-img-cmds.h: $(SRC_PATH)/qemu-img-cmds.hx
> >> $(call quiet-command,sh $(SRC_PATH)/hxtool -h < $< > $@," GEN
> $@")
> >> diff --git a/qemu-tool.c b/qemu-tool.c
> >> index 97ca949..8d6d0ef 100644
> >> --- a/qemu-tool.c
> >> +++ b/qemu-tool.c
> >> @@ -15,7 +15,6 @@
> >> #include "monitor.h"
> >> #include "qemu-timer.h"
> >> #include "qemu-log.h"
> >> -#include "qemu-error.h"
> >>
> >> #include <sys/time.h>
> >>
> >> @@ -33,6 +32,22 @@ void qemu_service_io(void)
> >> {
> >> }
> >>
> >> +Monitor *cur_mon;
> >> +
> >> +int monitor_cur_is_qmp(void)
> >> +{
> >> + return 0;
> >> +}
> >> +
> >> +void monitor_set_error(Monitor *mon, QError *qerror)
> >> +{
> >> + assert(0);
> >
> > Please use abort().
>
>
> Why?
Because assert(0) does not abort when compiled with -DNDEBUG.
- [Qemu-devel] [PATCH 1/6] error: Trim includes after "Move qemu_error & friends...", (continued)
[Qemu-devel] [PATCH 6/6] error: Move qerror_report() from qemu-error.[ch] to qerror.[ch], Markus Armbruster, 2010/03/17
[Qemu-devel] [PATCH 4/6] error: Make use of error_set_progname() optional, Markus Armbruster, 2010/03/17
[Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Markus Armbruster, 2010/03/17
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Blue Swirl, 2010/03/17
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Markus Armbruster, 2010/03/17
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o,
Blue Swirl <=
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Markus Armbruster, 2010/03/18
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Blue Swirl, 2010/03/18
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Markus Armbruster, 2010/03/18
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Blue Swirl, 2010/03/18
- Re: [Qemu-devel] [PATCH 5/6] error: Link qemu-img, qemu-nbd, qemu-io with qemu-error.o, Markus Armbruster, 2010/03/22
[Qemu-devel] [PATCH 2/6] error: Trim includes in qerror.c, Markus Armbruster, 2010/03/17