From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:37280) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T0x0R-0007k8-Cl for qemu-devel@nongnu.org; Mon, 13 Aug 2012 11:56:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1T0x0Q-00076D-6r for qemu-devel@nongnu.org; Mon, 13 Aug 2012 11:56:39 -0400 Received: from mnementh.archaic.org.uk ([81.2.115.146]:40944) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1T0x0P-00075x-WE for qemu-devel@nongnu.org; Mon, 13 Aug 2012 11:56:38 -0400 From: Peter Maydell Date: Mon, 13 Aug 2012 16:31:58 +0100 Message-Id: <1344871919-31559-10-git-send-email-peter.maydell@linaro.org> In-Reply-To: <1344871919-31559-1-git-send-email-peter.maydell@linaro.org> References: <1344871919-31559-1-git-send-email-peter.maydell@linaro.org> Subject: [Qemu-devel] [PATCH 09/10] ssd0323: abort() instead of exit(1) on error. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori , Blue Swirl Cc: qemu-devel@nongnu.org, Paul Brook From: "Peter A. G. Crosthwaite" To be more consistent with the newer ways of error signalling. That and SIGABT is easier to debug with than exit(1). Signed-off-by: Peter A. G. Crosthwaite Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- hw/ssd0323.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/hw/ssd0323.c b/hw/ssd0323.c index b0b2e94..b101c51 100644 --- a/hw/ssd0323.c +++ b/hw/ssd0323.c @@ -19,7 +19,9 @@ #define DPRINTF(fmt, ...) \ do { printf("ssd0323: " fmt , ## __VA_ARGS__); } while (0) #define BADF(fmt, ...) \ -do { fprintf(stderr, "ssd0323: error: " fmt , ## __VA_ARGS__); exit(1);} while (0) +do { \ + fprintf(stderr, "ssd0323: error: " fmt , ## __VA_ARGS__); abort(); \ +} while (0) #else #define DPRINTF(fmt, ...) do {} while(0) #define BADF(fmt, ...) \ -- 1.7.9.5