Merge pull request adafruit#2135 from jepler/audio-stuttering-background

tannewt · web-flow · commit 157429f20de8 · 2019-09-13T09:33:36.000-07:00
Reduce audio stuttering by running background tasks
diff --git a/py/obj.c b/py/obj.c
@@ -63,6 +63,9 @@ const char *mp_obj_get_type_str(mp_const_obj_t o_in) {
 void mp_obj_print_helper(const mp_print_t *print, mp_obj_t o_in, mp_print_kind_t kind) {
     // There can be data structures nested too deep, or just recursive
     MP_STACK_CHECK();
+    #ifdef RUN_BACKGROUND_TASKS
+    RUN_BACKGROUND_TASKS;
+    #endif
 #ifndef NDEBUG
     if (o_in == MP_OBJ_NULL) {
         mp_print_str(print, "(nil)");
diff --git a/py/stream.c b/py/stream.c
@@ -341,6 +341,9 @@ STATIC mp_obj_t stream_readall(mp_obj_t self_in) {
             p = vstr_extend(&vstr, DEFAULT_BUFFER_SIZE);
             current_read = DEFAULT_BUFFER_SIZE;
         }
+        #ifdef RUN_BACKGROUND_TASKS
+        RUN_BACKGROUND_TASKS;
+        #endif
     }
 
     vstr.len = total_size;
diff --git a/shared-module/os/__init__.c b/shared-module/os/__init__.c
@@ -128,6 +128,7 @@ mp_obj_t common_hal_os_listdir(const char* path) {
     while ((next = mp_iternext(iter_obj)) != MP_OBJ_STOP_ITERATION) {
         // next[0] is the filename.
         mp_obj_list_append(dir_list, mp_obj_subscr(next, MP_OBJ_NEW_SMALL_INT(0), MP_OBJ_SENTINEL));
+        RUN_BACKGROUND_TASKS;
     }
     return dir_list;
 }

Original file line number	Diff line number	Diff line change
`@@ -341,6 +341,9 @@ STATIC mp_obj_t stream_readall(mp_obj_t self_in) {`
`341`	`341`	`p = vstr_extend(&vstr, DEFAULT_BUFFER_SIZE);`
`342`	`342`	`current_read = DEFAULT_BUFFER_SIZE;`
`343`	`343`	`}`
	`344`	`+ #ifdef RUN_BACKGROUND_TASKS`
	`345`	`+ RUN_BACKGROUND_TASKS;`
	`346`	`+ #endif`
`344`	`347`	`}`
`345`	`348`
`346`	`349`	`vstr.len = total_size;`
Original file line number	Diff line number	Diff line change
`@@ -128,6 +128,7 @@ mp_obj_t common_hal_os_listdir(const char* path) {`
`128`	`128`	`while ((next = mp_iternext(iter_obj)) != MP_OBJ_STOP_ITERATION) {`
`129`	`129`	`// next[0] is the filename.`
`130`	`130`	`mp_obj_list_append(dir_list, mp_obj_subscr(next, MP_OBJ_NEW_SMALL_INT(0), MP_OBJ_SENTINEL));`
	`131`	`+ RUN_BACKGROUND_TASKS;`
`131`	`132`	`}`
`132`	`133`	`return dir_list;`
`133`	`134`	`}`