opcodes: blackfin: drop null/nul checks in OUTS

Parts of the disassembler rely on the disasm info never being NULL (such
as being able to read memory to disassemble in the first place).  So drop
useless null checks in the OUTS helper.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index c7d3cdf450b93b08ddac09099d21110287fbbc97..eb7fed9f4a068c04a52e39aecf8fbfd022501811 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,7 @@
+2011-02-22  Mike Frysinger  <vapier@gentoo.org>
+
+       * bfin-dis.c (OUTS): Remove p NULL check and txt NUL check.
+
 2011-02-22  Mike Frysinger  <vapier@gentoo.org>
 
        * bfin-dis.c (print_insn_bfin): Change outf->fprintf_func to OUTS.

diff --git a/opcodes/bfin-dis.c b/opcodes/bfin-dis.c
index 9b834a4130140251dd175de9299d4574e80f8f5d..2357a2a45de1561bec48673429eb5c97dc4531d5 100644 (file)
--- a/opcodes/bfin-dis.c
+++ b/opcodes/bfin-dis.c
@@ -490,7 +490,7 @@ static const enum machine_registers decode_allregs[] =
 
 /* (arch.pm)arch_disassembler_functions.  */
 #ifndef OUTS
-#define OUTS(p, txt) ((p) ? (((txt)[0]) ? (p->fprintf_func)(p->stream, "%s", txt) :0) :0)
+#define OUTS(p, txt) (p)->fprintf_func ((p)->stream, "%s", txt)
 #endif
 
 static void