From: Michael Buesch <mb@bu3sch.de>
Date: Sun, 19 Sep 2010 19:17:38 +0000 (+0200)
Subject: disassembler: Add error helpers
X-Git-Tag: b43-fwcutter-014~32
X-Git-Url: https://jxself.org/git/?a=commitdiff_plain;h=1a8718d7ef605cdbcd19965317f5758f1a8d5f1d;p=b43-tools.git

disassembler: Add error helpers

Signed-off-by: Michael Buesch <mb@bu3sch.de>
---

diff --git a/disassembler/main.c b/disassembler/main.c
index f833f8b..2e10012 100644
--- a/disassembler/main.c
+++ b/disassembler/main.c
@@ -68,6 +68,25 @@ const char *infile_name;
 const char *outfile_name;
 
 
+#define _msg_helper(type, msg, x...)	do {		\
+	fprintf(stderr, "Disassembler " type		\
+		":\n  " msg "\n" ,##x);			\
+					} while (0)
+
+#define dasm_error(msg, x...)	do {		\
+	_msg_helper("ERROR", msg ,##x);		\
+	exit(1);				\
+				} while (0)
+
+#define dasm_int_error(msg, x...) \
+	dasm_error("Internal error (bug): " msg ,##x)
+
+#define dasm_warn(msg, x...)	\
+	_msg_helper("warning", msg ,##x)
+
+#define asm_info(msg, x...)	\
+	_msg_helper("info", msg ,##x)
+
 static const char * gen_raw_code(unsigned int operand)
 {
 	char *ret;
@@ -103,8 +122,7 @@ static const char * disasm_indirect_mem_operand(unsigned int operand)
 		reg = ((operand >> 7) & 0x7);
 		break;
 	default:
-		fprintf(stderr, "Internal error: disasm_indirect_mem_operand invalid arch\n");
-		exit(1);
+		dasm_int_error("disasm_indirect_mem_operand invalid arch");
 	}
 	ret = xmalloc(12);
 	snprintf(ret, 12, "[0x%02X,off%u]", offset, reg);
@@ -128,8 +146,7 @@ static const char * disasm_imm_operand(unsigned int operand)
 		mask = 0x7FF;
 		break;
 	default:
-		fprintf(stderr, "Internal error: disasm_imm_operand invalid arch\n");
-		exit(1);
+		dasm_int_error("disasm_imm_operand invalid arch");
 	}
 
 	operand &= mask;
@@ -155,8 +172,7 @@ static const char * disasm_spr_operand(unsigned int operand)
 		mask = 0x7FF;
 		break;
 	default:
-		fprintf(stderr, "Internal error: disasm_spr_operand invalid arch\n");
-		exit(1);
+		dasm_int_error("disasm_spr_operand invalid arch");
 	}
 
 	ret = xmalloc(8);
@@ -178,8 +194,7 @@ static const char * disasm_gpr_operand(unsigned int operand)
 		mask = 0x7F;
 		break;
 	default:
-		fprintf(stderr, "Internal error: disasm_gpr_operand invalid arch\n");
-		exit(1);
+		dasm_int_error("disasm_gpr_operand invalid arch");
 	}
 
 	ret = xmalloc(5);
@@ -236,8 +251,7 @@ static void disasm_std_operand(struct statement *stmt,
 		}
 		break;
 	default:
-		fprintf(stderr, "Internal error: disasm_std_operand invalid arch\n");
-		exit(1);
+		dasm_int_error("disasm_std_operand invalid arch");
 	}
 raw:
 	stmt->u.insn.operands[out_idx] = gen_raw_code(operand);
@@ -493,24 +507,21 @@ static void disasm_constant_opcodes(struct disassembler_context *ctx,
 #if 0
 	case 0x004: {
 		if (cmdargs.arch != 15) {
-			fprintf(stderr, "Error: arch 15 call instruction found in arch %d binary\n",
-				cmdargs.arch);
-			exit(1);
+			dasm_error("arch 15 call instruction found in arch %d binary",
+				   cmdargs.arch);
 		}
 		stmt->u.insn.name = "call";
 		stmt->u.insn.is_labelref = 0;
 		stmt->u.insn.labeladdr = stmt->u.insn.bin->operands[2];
 		if (stmt->u.insn.bin->operands[0] != 0x1780 ||
-		    stmt->u.insn.bin->operands[1] != 0x1780) {
-			fprintf(stderr, "r15 call: Invalid first or second argument\n");
-		}
+		    stmt->u.insn.bin->operands[1] != 0x1780)
+			dasm_warn("r15 call: Invalid first or second argument");
 		break;
 	}
 	case 0x005: {
 		if (cmdargs.arch != 15) {
-			fprintf(stderr, "Error: arch 15 ret instruction found in arch %d binary\n",
-				cmdargs.arch);
-			exit(1);
+			dasm_error("arch 15 ret instruction found in arch %d binary",
+				   cmdargs.arch);
 		}
 		stmt->u.insn.name = "ret";
 		break;
@@ -527,8 +538,7 @@ static void disasm_constant_opcodes(struct disassembler_context *ctx,
 			mask = 0x7FF;
 			break;
 		default:
-			fprintf(stderr, "Internal error: TKIP invalid arch\n");
-			exit(1);
+			dasm_int_error("TKIP invalid arch");
 		}
 
 		flags = stmt->u.insn.bin->operands[1];
@@ -546,9 +556,7 @@ static void disasm_constant_opcodes(struct disassembler_context *ctx,
 			stmt->u.insn.name = "tkipls";
 			break;
 		default:
-			fprintf(stderr, "Invalid TKIP flags %X\n",
-				flags);
-			exit(1);
+			dasm_error("Invalid TKIP flags %X", flags);
 		}
 		disasm_std_operand(stmt, 0, 0, 0);
 		disasm_std_operand(stmt, 2, 2, 0);
@@ -566,20 +574,19 @@ static void disasm_constant_opcodes(struct disassembler_context *ctx,
 			mask = 0x1780;
 			break;
 		default:
-			fprintf(stderr, "Internal error: NAP invalid arch\n");
-			exit(1);
+			dasm_int_error("NAP invalid arch");
 		}
 		if (stmt->u.insn.bin->operands[0] != mask) {
-			fprintf(stderr, "NAP: invalid first argument 0x%04X\n",
-				stmt->u.insn.bin->operands[0]);
+			dasm_warn("NAP: invalid first argument 0x%04X\n",
+				  stmt->u.insn.bin->operands[0]);
 		}
 		if (stmt->u.insn.bin->operands[1] != mask) {
-			fprintf(stderr, "NAP: invalid second argument 0x%04X\n",
-				stmt->u.insn.bin->operands[1]);
+			dasm_warn("NAP: invalid second argument 0x%04X\n",
+				  stmt->u.insn.bin->operands[1]);
 		}
 		if (stmt->u.insn.bin->operands[2] != 0) {
-			fprintf(stderr, "NAP: invalid third argument 0x%04X\n",
-				stmt->u.insn.bin->operands[2]);
+			dasm_warn("NAP: invalid third argument 0x%04X\n",
+				  stmt->u.insn.bin->operands[2]);
 		}
 		break;
 	}
@@ -747,11 +754,8 @@ static void resolve_labels(struct disassembler_context *ctx)
 			continue;
 		labeladdr = stmt->u.insn.labeladdr;
 		label = get_label_at(ctx, labeladdr);
-		if (!label) {
-			fprintf(stderr, "Labeladdress %X out of bounds\n",
-				labeladdr);
-			exit(1);
-		}
+		if (!label)
+			dasm_error("Labeladdress %X out of bounds", labeladdr);
 		stmt->u.insn.labelref = label;
 	}