From bd83074ab37f96455a2aff35c37ef6c3f07ebaec Mon Sep 17 00:00:00 2001
From: Jordan <jordanlavatai@gmail.com>
Date: Mon, 16 Jan 2017 14:50:43 -0800
Subject: [PATCH]  added dprintf macro, finally

---
 src/lexer.rl | 41 ++++++++++++++++++++++-------------------
 1 file changed, 22 insertions(+), 19 deletions(-)

diff --git a/src/lexer.rl b/src/lexer.rl
index da397c0..7f65276 100644
--- a/src/lexer.rl
+++ b/src/lexer.rl
@@ -28,7 +28,7 @@ uint8_t   lval_offs;
     PUSHTOK(T,lval_stack + lval_offs);		\
     lval_offs++;				\
     ntok++;					\
-  } while (0);
+  } while (0)
 #define PUSHFACE(F) LEXTOK(FACING, face, F)
 #define PUSHREF(R)  LEXTOK(REF, ref, R)
 #define PUSHLINK()  LEXTOK(LINK, val, 0)
@@ -38,6 +38,10 @@ uint8_t   lval_offs;
 #define PUSHPATH(P) LEXTOK(PATH, str, P)
 
 #define DEBUG 1
+#define dprintf(_B, ...) do {	        \
+    if(_B) printf(__VA_ARGS__);	\
+  } while (0) 
+  
 
 /* Lexstring is the main lexer for APC and is generated by ragel. It lexes file names of files 
    that have been scanned and pushes their types and values into the tok_stack, which yyparse 
@@ -64,45 +68,45 @@ uint8_t   lval_offs;
 			    }
 			  PUSHNUM(lval.val);
                         }
-  action push_name      { if(DEBUG) printf("Lexer_lexstring:: action:push_name: from %s to %s\n", ts, p);
+  action push_name      { dprintf(DEBUG, "Lexer_lexstring:: action:push_name: from %s to %s\n", ts, p);
 			  PUSHNAME(ts);
                         }
-  action push_map       { if(DEBUG) printf("Lexer_lexstring:: action:push_map: pushing map token\n");
+  action push_map       { dprintf(DEBUG, "Lexer_lexstring:: action:push_map: pushing map token\n");
 			  PUSHOP(MAP);
 		        }
-  action set_ts         { if(DEBUG) printf("Lexer_lexstring:: action:set_ts. ts = %s\n", p);
+  action set_ts         { dprintf(DEBUG, "Lexer_lexstring:: action:set_ts. ts = %s\n", p);
                           ts = p; }
-  action push_SS        { if(DEBUG) printf("Lexer_lexstring:: action:push_SS. p = %s\n",p);
+  action push_SS        { dprintf(DEBUG, "Lexer_lexstring:: action:push_SS. p = %s\n",p);
                           PUSHOP(SS);
                         }
-  action push_S         { if(DEBUG) printf("Lexer_lexstring:: action:push_S. p = %s\n", p);
+  action push_S         { dprintf(DEBUG, "Lexer_lexstring:: action:push_S. p = %s\n", p);
 			  PUSHFACE(SFACE);
                         }
-  action push_SW        { if(DEBUG) printf("Lexer_lexstring:: action:push_SW. p = %s\n", p);
+  action push_SW        { dprintf(DEBUG, "Lexer_lexstring:: action:push_SW. p = %s\n", p);
                           PUSHFACE(SWFACE);
 		        }
-  action push_W         { if(DEBUG) printf("Lexer_lexstring:: action:push_W. p = %s\n", p);
+  action push_W         { dprintf(DEBUG, "Lexer_lexstring:: action:push_W. p = %s\n", p);
                           PUSHFACE(WFACE);
                         }
-  action push_NW        { if(DEBUG) printf("Lexer_lexstring:: action:push_NW. p = %s\n", p);
+  action push_NW        { dprintf(DEBUG, "Lexer_lexstring:: action:push_NW. p = %s\n", p);
 			  PUSHFACE(NWFACE);
                         }
-  action push_N         { if(DEBUG) printf("Lexer_lexstring:: action:push_N. p = %s\n", p);
+  action push_N         { dprintf(DEBUG, "Lexer_lexstring:: action:push_N. p = %s\n", p);
                           PUSHFACE(NFACE);
                         }
-  action push_NE        { if(DEBUG) printf("Lexer_lexstring:: action:push_NE. p = %s\n", p);
+  action push_NE        { dprintf(DEBUG, "Lexer_lexstring:: action:push_NE. p = %s\n", p);
 			  PUSHFACE(NEFACE);
                         }
-  action push_E         { if(DEBUG) printf("Lexer_lexstring:: action:push_N. p = %s\n", p);
+  action push_E         { dprintf(DEBUG, "Lexer_lexstring:: action:push_N. p = %s\n", p);
 			  PUSHFACE(EFACE);
                         }
-  action push_SE        { if(DEBUG) printf("Lexer_lexstring:: action:push_N. p = %s\n", p);
+  action push_SE        { dprintf(DEBUG, "Lexer_lexstring:: action:push_N. p = %s\n", p);
 			  PUSHFACE(SEFACE);
                         }
-  action ref_error      { if(DEBUG) printf("ref from %s to %s has an inappropriate amount of hex digits, it must have eight.\n", ts, p);
+  action ref_error      { dprintf(DEBUG, "ref from %s to %s has an inappropriate amount of hex digits, it must have eight.\n", ts, p);
                           exit(1);
                         }
-  action p              { if(DEBUG) printf("Lexer_lexstring:: p = %s\n", p);
+  action p              { dprintf("Lexer_lexstring:: p = %s\n", p);
                         }
     
   N = 'N' %push_N;
@@ -151,8 +155,7 @@ int lexer_lexstring
 ( uint8_t* str, 
   int size
 )
-{ uint8_t *p; 
-  uint8_t *ts, *pe, *eof;
+{ uint8_t* p, * ts, * pe, * eof;
   int cs, ntok;
   YYSTYPE lval;
 
@@ -160,12 +163,12 @@ int lexer_lexstring
   p = ts = str;
   pe = eof =  p + size + 1;
 
-  if(DEBUG) printf("|---Begin lexstring on p = %s, pe = %s.\n",p, pe);
+  dprintf("|---Begin lexstring on p = %s, pe = %s.\n", (char*)p, pe);
 
   %%write init;
   %%write exec;
 
-  if(DEBUG) printf("Ending lexstring of file %s, pushed %d tokens.\n",str, ntok);
+  dprintf("Ending lexstring of file %s, pushed %d tokens.\n", (char*)str, ntok);
 
   return ntok;
 }
-- 
2.18.0