From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:32842) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QZMA9-0007mR-Oy for qemu-devel@nongnu.org; Wed, 22 Jun 2011 08:04:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QZMA7-0005Ua-P6 for qemu-devel@nongnu.org; Wed, 22 Jun 2011 08:04:05 -0400 Received: from oxygen.pond.sub.org ([78.46.104.156]:50619) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QZMA7-0005Tp-02 for qemu-devel@nongnu.org; Wed, 22 Jun 2011 08:04:03 -0400 From: Markus Armbruster Date: Wed, 22 Jun 2011 14:03:52 +0200 Message-Id: <1308744237-3468-1-git-send-email-armbru@redhat.com> Subject: [Qemu-devel] [PATCH trivial 0/5] Fix error_report() calls List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: stefanha@gmail.com I limited myself to truly trivial fixes. I hope that's still useful enough to excuse a little rant. "bug: 0 0" is not a useful error message. Neither are "Desc next is 128", " too short" (too short indeed), "wrong magic" (some "wrong magic" skills will be handy to figure out what went wrong). I'm stopping here for brevity's sake, not for want of examples. If you can't be bothered to provide an intelligible error message and *test* it to verify it actually makes sense for the user, maybe you should stick to assert() or something. Makes your laziness stand out properly. Markus Armbruster (5): sheepdog: qemu_bh_new() can't return null pointer, drop check Strip trailing '\n' from error_report()'s first argument qemu-img: Don't prepend qemu-img to error messages twice. Spell "unkown" correctly in error_report() arguments blockdev: Put space after comma in error message block-migration.c | 2 +- block/qcow2.c | 2 +- block/sheepdog.c | 90 ++++++++++++++++++++++------------------------- blockdev.c | 2 +- hw/lm32_sys.c | 2 +- hw/lm32_timer.c | 4 +- hw/lm32_uart.c | 4 +- hw/milkymist-ac97.c | 4 +- hw/milkymist-memcard.c | 4 +- hw/milkymist-minimac2.c | 6 ++-- hw/milkymist-pfpu.c | 4 +- hw/milkymist-softusb.c | 8 ++-- hw/milkymist-sysctl.c | 4 +- hw/milkymist-tmu2.c | 8 ++-- hw/qdev.c | 2 +- hw/strongarm.c | 4 +- hw/usb-ccid.c | 2 +- hw/virtio-serial-bus.c | 10 +++--- hw/virtio.c | 4 +- net.c | 2 +- net/socket.c | 6 ++-- qemu-error.c | 2 + qemu-img.c | 12 +++--- 23 files changed, 92 insertions(+), 96 deletions(-) -- 1.7.2.3