py/stream: Remove unnecessary check for NULL return from vstr_extend.

dpgeorge · dpgeorge · commit 79ec869f9593 · 2016-09-22T10:50:47.000+10:00
vstr_extend will now only return NULL if the vstr is a fixed buffer, which
in this case it is not.
diff --git a/py/stream.c b/py/stream.c
@@ -355,10 +355,6 @@ STATIC mp_obj_t stream_readall(mp_obj_t self_in) {
         } else {
             p = vstr_extend(&vstr, DEFAULT_BUFFER_SIZE);
             current_read = DEFAULT_BUFFER_SIZE;
-            if (p == NULL) {
-                // TODO
-                nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_OSError/*&mp_type_RuntimeError*/, "Out of memory"));
-            }
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -355,10 +355,6 @@ STATIC mp_obj_t stream_readall(mp_obj_t self_in) {`
`355`	`355`	`} else {`
`356`	`356`	`p = vstr_extend(&vstr, DEFAULT_BUFFER_SIZE);`
`357`	`357`	`current_read = DEFAULT_BUFFER_SIZE;`
`358`		`- if (p == NULL) {`
`359`		`- // TODO`
`360`		`- nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_OSError/&mp_type_RuntimeError/, "Out of memory"));`
`361`		`- }`
`362`	`358`	`}`
`363`	`359`	`}`
`364`	`360`