comment out dead code in Opcode::
authorDaniel Dragan <bulk88@hotmail.com>
Tue, 20 Jan 2015 05:16:57 +0000 (00:16 -0500)
committerTony Cook <tony@develop-help.com>
Wed, 28 Jan 2015 23:21:28 +0000 (10:21 +1100)
opcode_debug has never had an API to turn it on/set it to non-0 since
Opcode::'s initial commit 6badd1a5d1 in 5.003001. Making opcode_debug a
constant allows the CC to constant fold away the code, the warn string
literals, and makes the my_cxt_t struct slightly smaller. Dont remove the
code entirely since someone might find it useful one day.

ext/Opcode/Opcode.xs

index 3b6d275..c1170e9 100644 (file)
@@ -13,7 +13,9 @@ typedef struct {
     HV *       x_op_named_bits;        /* cache shared for whole process */
     SV *       x_opset_all;            /* mask with all bits set       */
     IV         x_opset_len;            /* length of opmasks in bytes   */
-    int                x_opcode_debug;
+#if 0
+    int                x_opcode_debug;         /* unused warn() emitting debugging code */
+#endif
 } my_cxt_t;
 
 START_MY_CXT
@@ -21,7 +23,12 @@ START_MY_CXT
 #define op_named_bits          (MY_CXT.x_op_named_bits)
 #define opset_all              (MY_CXT.x_opset_all)
 #define opset_len              (MY_CXT.x_opset_len)
-#define opcode_debug           (MY_CXT.x_opcode_debug)
+#if 0
+#  define opcode_debug         (MY_CXT.x_opcode_debug)
+#else
+ /* no API to turn this on at runtime, so constant fold the code away */
+#  define opcode_debug         0
+#endif
 
 static SV  *new_opset (pTHX_ SV *old_opset);
 static int  verify_opset (pTHX_ SV *opset, int fatal);