From 7860c2a68acd1ccd28cb4bb87819487fd6966a9e Mon Sep 17 00:00:00 2001
From: Damien George <damien.p.george@gmail.com>
Date: Wed, 5 Nov 2014 21:16:41 +0000
Subject: [PATCH] py: Fix some macros defines; cleanup some includes.

---
 py/bc.c            | 1 +
 py/builtinimport.c | 1 +
 py/gc.c            | 3 +++
 py/map.c           | 2 +-
 py/mpconfig.h      | 4 ++--
 py/nlr.h           | 2 +-
 py/nlrthumb.S      | 4 ++--
 py/objfun.c        | 1 +
 py/objtype.c       | 1 +
 stmhal/printf.c    | 7 -------
 stmhal/pybstdio.h  | 2 +-
 11 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/py/bc.c b/py/bc.c
index b5515bcd7..47f8d9f71 100644
--- a/py/bc.c
+++ b/py/bc.c
@@ -44,6 +44,7 @@
 #if 0 // print debugging info
 #define DEBUG_PRINT (1)
 #else // don't print debugging info
+#define DEBUG_PRINT (0)
 #define DEBUG_printf(...) (void)0
 #endif
 
diff --git a/py/builtinimport.c b/py/builtinimport.c
index af93aace9..f1288744a 100644
--- a/py/builtinimport.c
+++ b/py/builtinimport.c
@@ -50,6 +50,7 @@
 #define DEBUG_PRINT (1)
 #define DEBUG_printf DEBUG_printf
 #else // don't print debugging info
+#define DEBUG_PRINT (0)
 #define DEBUG_printf(...) (void)0
 #endif
 
diff --git a/py/gc.c b/py/gc.c
index fca5050e7..4d9334056 100644
--- a/py/gc.c
+++ b/py/gc.c
@@ -44,6 +44,7 @@
 #define DEBUG_PRINT (1)
 #define DEBUG_printf DEBUG_printf
 #else // don't print debugging info
+#define DEBUG_PRINT (0)
 #define DEBUG_printf(...) (void)0
 #endif
 
@@ -746,7 +747,9 @@ void gc_dump_alloc_table(void) {
                 if (*ptr == (mp_uint_t)&mp_type_tuple) { c = 'T'; }
                 else if (*ptr == (mp_uint_t)&mp_type_list) { c = 'L'; }
                 else if (*ptr == (mp_uint_t)&mp_type_dict) { c = 'D'; }
+                #if MICROPY_PY_BUILTINS_FLOAT
                 else if (*ptr == (mp_uint_t)&mp_type_float) { c = 'F'; }
+                #endif
                 else if (*ptr == (mp_uint_t)&mp_type_fun_bc) { c = 'B'; }
                 else if (*ptr == (mp_uint_t)&mp_type_module) { c = 'M'; }
                 else { c = 'h'; }
diff --git a/py/map.c b/py/map.c
index 906a1611f..fb0c5a079 100644
--- a/py/map.c
+++ b/py/map.c
@@ -358,7 +358,7 @@ void mp_set_clear(mp_set_t *set) {
     set->table = NULL;
 }
 
-#if DEBUG_PRINT
+#if defined(DEBUG_PRINT) && DEBUG_PRINT
 void mp_map_dump(mp_map_t *map) {
     for (mp_uint_t i = 0; i < map->alloc; i++) {
         if (map->table[i].key != NULL) {
diff --git a/py/mpconfig.h b/py/mpconfig.h
index 8ef3e62fe..02c6674d8 100644
--- a/py/mpconfig.h
+++ b/py/mpconfig.h
@@ -184,8 +184,8 @@
 #endif
 
 // Whether to enable finalisers in the garbage collector (ie call __del__)
-#ifndef MICROPY_ENABLE_GC_FINALISER
-#define MICROPY_ENABLE_GC_FINALISER (0)
+#ifndef MICROPY_ENABLE_FINALISER
+#define MICROPY_ENABLE_FINALISER (0)
 #endif
 
 // Whether to check C stack usage. C stack used for calling Python functions,
diff --git a/py/nlr.h b/py/nlr.h
index cb2b53b85..32e4b7cb2 100644
--- a/py/nlr.h
+++ b/py/nlr.h
@@ -36,7 +36,7 @@ struct _nlr_buf_t {
     // the entries here must all be machine word size
     nlr_buf_t *prev;
     void *ret_val;
-#if !MICROPY_NLR_SETJMP
+#if !defined(MICROPY_NLR_SETJMP) || !MICROPY_NLR_SETJMP
 #if defined(__i386__)
     void *regs[6];
 #elif defined(__x86_64__)
diff --git a/py/nlrthumb.S b/py/nlrthumb.S
index 5443edfe8..dcbcc2770 100644
--- a/py/nlrthumb.S
+++ b/py/nlrthumb.S
@@ -24,7 +24,7 @@
  * THE SOFTWARE.
  */
 
-#if !MICROPY_NLR_SETJMP && (defined(__thumb2__) || defined(__thumb__) || defined(__arm__))
+#if (!defined(MICROPY_NLR_SETJMP) || !MICROPY_NLR_SETJMP) && (defined(__thumb2__) || defined(__thumb__) || defined(__arm__))
 
 // We only need the functions here if we are on arm/thumb, and we are not
 // using setjmp/longjmp.
@@ -138,4 +138,4 @@ nlr_jump:
 nlr_top:
     .space  4
 
-#endif // !MICROPY_NLR_SETJMP && (defined(__thumb2__) || defined(__thumb__) || defined(__arm__))
+#endif // (!defined(MICROPY_NLR_SETJMP) || !MICROPY_NLR_SETJMP) && (defined(__thumb2__) || defined(__thumb__) || defined(__arm__))
diff --git a/py/objfun.c b/py/objfun.c
index 3adf05e3b..5f95a015b 100644
--- a/py/objfun.c
+++ b/py/objfun.c
@@ -44,6 +44,7 @@
 #if 0 // print debugging info
 #define DEBUG_PRINT (1)
 #else // don't print debugging info
+#define DEBUG_PRINT (0)
 #define DEBUG_printf(...) (void)0
 #endif
 
diff --git a/py/objtype.c b/py/objtype.c
index 171ea838f..70dc226a6 100644
--- a/py/objtype.c
+++ b/py/objtype.c
@@ -43,6 +43,7 @@
 #define DEBUG_PRINT (1)
 #define DEBUG_printf DEBUG_printf
 #else // don't print debugging info
+#define DEBUG_PRINT (0)
 #define DEBUG_printf(...) (void)0
 #endif
 
diff --git a/stmhal/printf.c b/stmhal/printf.c
index 137189b94..ca9b279cd 100644
--- a/stmhal/printf.c
+++ b/stmhal/printf.c
@@ -29,17 +29,10 @@
 #include <stdarg.h>
 
 #include "mpconfig.h"
-#include "std.h"
 #include "misc.h"
-#include "systick.h"
 #include "qstr.h"
 #include "obj.h"
 #include "pfenv.h"
-#if 0
-#include "lcd.h"
-#endif
-#include "uart.h"
-#include "usb.h"
 #include "pybstdio.h"
 
 #if MICROPY_PY_BUILTINS_FLOAT
diff --git a/stmhal/pybstdio.h b/stmhal/pybstdio.h
index cdb7acfe5..08a2d0363 100644
--- a/stmhal/pybstdio.h
+++ b/stmhal/pybstdio.h
@@ -24,7 +24,7 @@
  * THE SOFTWARE.
  */
 
-extern pyb_uart_obj_t *pyb_stdio_uart;
+extern struct _pyb_uart_obj_t *pyb_stdio_uart;
 
 void stdout_tx_str(const char *str);
 void stdout_tx_strn(const char *str, mp_uint_t len);
-- 
GitLab