commit 5ae5ec986ea5e14321f6c8091454c165c23b40c5
author Damien George <damien.p.george@gmail.com> Sat Apr 11 12:01:39 2015 +0100
committer Damien George <damien.p.george@gmail.com> Thu Apr 16 14:30:16 2015 +0000
tree 65c080e594f9268fc6d0efea7afed956e14dded8
parent 7f9d1d6ab923096582622b700bedb6a571518eac

py: Make mp_sys_stdout_print object, wrapping sys.stdout for mp_print*.

So now all printing should go via either mp_plat_print or
mp_sys_stdout_print.

py/modbuiltins.c

diff --git a/py/modbuiltins.c b/py/modbuiltins.c
index f080f45..f001d70 100644
--- a/py/modbuiltins.c
+++ b/py/modbuiltins.c
@@ -440,9 +440,8 @@
 STATIC mp_obj_t mp_builtin___repl_print__(mp_obj_t o) {
     if (o != mp_const_none) {
         #if MICROPY_PY_IO
-        mp_print_t print = {&mp_sys_stdout_obj, (mp_print_strn_t)mp_stream_write};
-        mp_obj_print_helper(&print, o, PRINT_REPR);
-        mp_stream_write(&mp_sys_stdout_obj, "\n", 1);
+        mp_obj_print_helper(&mp_sys_stdout_print, o, PRINT_REPR);
+        mp_print_str(&mp_sys_stdout_print, "\n");
         #else
         mp_obj_print(o, PRINT_REPR);
         printf("\n");

py/modsys.c

diff --git a/py/modsys.c b/py/modsys.c
index 09e72e9..537befd 100644
--- a/py/modsys.c
+++ b/py/modsys.c
@@ -42,6 +42,10 @@
 extern struct _mp_dummy_t mp_sys_stdout_obj;
 extern struct _mp_dummy_t mp_sys_stderr_obj;
 
+#if MICROPY_PY_IO
+const mp_print_t mp_sys_stdout_print = {&mp_sys_stdout_obj, (mp_print_strn_t)mp_stream_write};
+#endif
+
 /// \constant version - Python language version that this implementation conforms to, as a string
 STATIC const MP_DEFINE_STR_OBJ(version_obj, "3.4.0");
 

py/mpprint.h

diff --git a/py/mpprint.h b/py/mpprint.h
index 60fa18a..de497ce 100644
--- a/py/mpprint.h
+++ b/py/mpprint.h
@@ -47,10 +47,13 @@
     mp_print_strn_t print_strn;
 } mp_print_t;
 
-// Wrapper for platform print function, which wraps MP_PLAT_PRINT_STRN.
-// All (non-debug) prints go through this interface (except some which
-// go through mp_sys_stdout_obj if MICROPY_PY_IO is defined).
+// All (non-debug) prints go through one of the two interfaces below.
+// 1) Wrapper for platform print function, which wraps MP_PLAT_PRINT_STRN.
 extern const mp_print_t mp_plat_print;
+#if MICROPY_PY_IO
+// 2) Wrapper for printing to sys.stdout.
+extern const mp_print_t mp_sys_stdout_print;
+#endif
 
 int mp_print_str(const mp_print_t *print, const char *str);
 int mp_print_strn(const mp_print_t *print, const char *str, mp_uint_t len, int flags, char fill, int width);

py/obj.c

diff --git a/py/obj.c b/py/obj.c
index 0d318fa..eb9af9e 100644
--- a/py/obj.c
+++ b/py/obj.c
@@ -73,12 +73,7 @@
 
 void mp_obj_print(mp_obj_t o_in, mp_print_kind_t kind) {
 #if MICROPY_PY_IO
-    // defined per port; type of these is irrelevant, just need pointer
-    extern struct _mp_dummy_t mp_sys_stdout_obj;
-    mp_print_t print;
-    print.data = &mp_sys_stdout_obj;
-    print.print_strn = (mp_print_strn_t)mp_stream_write;
-    mp_obj_print_helper(&print, o_in, kind);
+    mp_obj_print_helper(&mp_sys_stdout_print, o_in, kind);
 #else
     mp_obj_print_helper(&mp_plat_print, o_in, kind);
 #endif