Skip to content

Commit 8a0ee5a

Browse files
jimmodpgeorge
jimmo
authored and
dpgeorge
committed
py/objstr: Split mp_obj_str_from_vstr into bytes/str versions.
Previously the desired output type was specified. Now make the type part of the function name. Because this function is used in a few places this saves code size due to smaller call-site. This makes `mp_obj_new_str_type_from_vstr` a private function of objstr.c (which is almost the only place where the output type isn't a compile-time constant). This saves ~140 bytes on PYBV11. Signed-off-by: Jim Mussared <[email protected]>
1 parent 09879f9 commit 8a0ee5a

37 files changed

+102
-73
lines changed

‎extmod/machine_i2c.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -417,7 +417,7 @@ STATIC mp_obj_t machine_i2c_readfrom(size_t n_args, const mp_obj_t *args) {
417417
if (ret < 0) {
418418
mp_raise_OSError(-ret);
419419
}
420-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
420+
return mp_obj_new_bytes_from_vstr(&vstr);
421421
}
422422
MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(machine_i2c_readfrom_obj, 3, 4, machine_i2c_readfrom);
423423

@@ -581,7 +581,7 @@ STATIC mp_obj_t machine_i2c_readfrom_mem(size_t n_args, const mp_obj_t *pos_args
581581
mp_raise_OSError(-ret);
582582
}
583583

584-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
584+
return mp_obj_new_bytes_from_vstr(&vstr);
585585
}
586586
MP_DEFINE_CONST_FUN_OBJ_KW(machine_i2c_readfrom_mem_obj, 1, machine_i2c_readfrom_mem);
587587

‎extmod/machine_spi.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ STATIC mp_obj_t mp_machine_spi_read(size_t n_args, const mp_obj_t *args) {
7070
vstr_init_len(&vstr, mp_obj_get_int(args[1]));
7171
memset(vstr.buf, n_args == 3 ? mp_obj_get_int(args[2]) : 0, vstr.len);
7272
mp_machine_spi_transfer(args[0], vstr.len, vstr.buf, vstr.buf);
73-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
73+
return mp_obj_new_bytes_from_vstr(&vstr);
7474
}
7575
MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(mp_machine_spi_read_obj, 2, 3, mp_machine_spi_read);
7676

‎extmod/modlwip.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -1198,7 +1198,7 @@ STATIC mp_obj_t lwip_socket_recv(mp_obj_t self_in, mp_obj_t len_in) {
11981198
return mp_const_empty_bytes;
11991199
}
12001200
vstr.len = ret;
1201-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
1201+
return mp_obj_new_bytes_from_vstr(&vstr);
12021202
}
12031203
STATIC MP_DEFINE_CONST_FUN_OBJ_2(lwip_socket_recv_obj, lwip_socket_recv);
12041204

@@ -1271,7 +1271,7 @@ STATIC mp_obj_t lwip_socket_recvfrom(mp_obj_t self_in, mp_obj_t len_in) {
12711271
tuple[0] = mp_const_empty_bytes;
12721272
} else {
12731273
vstr.len = ret;
1274-
tuple[0] = mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
1274+
tuple[0] = mp_obj_new_bytes_from_vstr(&vstr);
12751275
}
12761276
tuple[1] = netutils_format_inet_addr(ip, port, NETUTILS_BIG);
12771277
return mp_obj_new_tuple(2, tuple);

‎extmod/modubinascii.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ STATIC mp_obj_t mod_binascii_a2b_base64(mp_obj_t data) {
104104
mp_raise_ValueError(MP_ERROR_TEXT("incorrect padding"));
105105
}
106106

107-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
107+
return mp_obj_new_bytes_from_vstr(&vstr);
108108
}
109109
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mod_binascii_a2b_base64_obj, mod_binascii_a2b_base64);
110110

@@ -166,7 +166,7 @@ STATIC mp_obj_t mod_binascii_b2a_base64(size_t n_args, const mp_obj_t *pos_args,
166166
if (newline) {
167167
*out = '\n';
168168
}
169-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
169+
return mp_obj_new_bytes_from_vstr(&vstr);
170170
}
171171
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(mod_binascii_b2a_base64_obj, 1, mod_binascii_b2a_base64);
172172

‎extmod/moducryptolib.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@ STATIC mp_obj_t aes_process(size_t n_args, const mp_obj_t *args, bool encrypt) {
329329
if (out_buf != MP_OBJ_NULL) {
330330
return out_buf;
331331
}
332-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
332+
return mp_obj_new_bytes_from_vstr(&vstr);
333333
}
334334

335335
STATIC mp_obj_t ucryptolib_aes_encrypt(size_t n_args, const mp_obj_t *args) {

‎extmod/moduhashlib.c

+6-6
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ STATIC mp_obj_t uhashlib_sha256_digest(mp_obj_t self_in) {
109109
vstr_t vstr;
110110
vstr_init_len(&vstr, 32);
111111
mbedtls_sha256_finish_ret((mbedtls_sha256_context *)&self->state, (unsigned char *)vstr.buf);
112-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
112+
return mp_obj_new_bytes_from_vstr(&vstr);
113113
}
114114

115115
#else
@@ -143,7 +143,7 @@ STATIC mp_obj_t uhashlib_sha256_digest(mp_obj_t self_in) {
143143
vstr_t vstr;
144144
vstr_init_len(&vstr, SHA256_BLOCK_SIZE);
145145
sha256_final((CRYAL_SHA256_CTX *)self->state, (byte *)vstr.buf);
146-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
146+
return mp_obj_new_bytes_from_vstr(&vstr);
147147
}
148148
#endif
149149

@@ -196,7 +196,7 @@ STATIC mp_obj_t uhashlib_sha1_digest(mp_obj_t self_in) {
196196
vstr_t vstr;
197197
vstr_init_len(&vstr, SHA1_SIZE);
198198
SHA1_Final((byte *)vstr.buf, (SHA1_CTX *)self->state);
199-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
199+
return mp_obj_new_bytes_from_vstr(&vstr);
200200
}
201201
#endif
202202

@@ -237,7 +237,7 @@ STATIC mp_obj_t uhashlib_sha1_digest(mp_obj_t self_in) {
237237
vstr_init_len(&vstr, 20);
238238
mbedtls_sha1_finish_ret((mbedtls_sha1_context *)self->state, (byte *)vstr.buf);
239239
mbedtls_sha1_free((mbedtls_sha1_context *)self->state);
240-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
240+
return mp_obj_new_bytes_from_vstr(&vstr);
241241
}
242242
#endif
243243

@@ -289,7 +289,7 @@ STATIC mp_obj_t uhashlib_md5_digest(mp_obj_t self_in) {
289289
vstr_t vstr;
290290
vstr_init_len(&vstr, MD5_SIZE);
291291
MD5_Final((byte *)vstr.buf, (MD5_CTX *)self->state);
292-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
292+
return mp_obj_new_bytes_from_vstr(&vstr);
293293
}
294294
#endif // MICROPY_SSL_AXTLS
295295

@@ -330,7 +330,7 @@ STATIC mp_obj_t uhashlib_md5_digest(mp_obj_t self_in) {
330330
vstr_init_len(&vstr, 16);
331331
mbedtls_md5_finish_ret((mbedtls_md5_context *)self->state, (byte *)vstr.buf);
332332
mbedtls_md5_free((mbedtls_md5_context *)self->state);
333-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
333+
return mp_obj_new_bytes_from_vstr(&vstr);
334334
}
335335
#endif // MICROPY_SSL_MBEDTLS
336336

‎extmod/modujson.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ STATIC mp_obj_t mod_ujson_dump_helper(size_t n_args, const mp_obj_t *pos_args, m
6767
vstr_t vstr;
6868
vstr_init_print(&vstr, 8, &print_ext.base);
6969
mp_obj_print_helper(&print_ext.base, pos_args[0], PRINT_JSON);
70-
return mp_obj_new_str_from_vstr(&mp_type_str, &vstr);
70+
return mp_obj_new_str_from_vstr(&vstr);
7171
} else {
7272
// dump(obj, stream)
7373
print_ext.base.data = MP_OBJ_TO_PTR(pos_args[1]);
@@ -103,7 +103,7 @@ STATIC mp_obj_t mod_ujson_dumps(mp_obj_t obj) {
103103
mp_print_t print;
104104
vstr_init_print(&vstr, 8, &print);
105105
mp_obj_print_helper(&print, obj, PRINT_JSON);
106-
return mp_obj_new_str_from_vstr(&mp_type_str, &vstr);
106+
return mp_obj_new_str_from_vstr(&vstr);
107107
}
108108
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mod_ujson_dumps_obj, mod_ujson_dumps);
109109

‎extmod/modure.c

+5-1
Original file line numberDiff line numberDiff line change
@@ -372,7 +372,11 @@ STATIC mp_obj_t re_sub_helper(size_t n_args, const mp_obj_t *args) {
372372
// Add post-match string
373373
vstr_add_strn(&vstr_return, subj.begin, subj.end - subj.begin);
374374

375-
return mp_obj_new_str_from_vstr(mp_obj_get_type(where), &vstr_return);
375+
if (mp_obj_get_type(where) == &mp_type_str) {
376+
return mp_obj_new_str_from_vstr(&vstr_return);
377+
} else {
378+
return mp_obj_new_bytes_from_vstr(&vstr_return);
379+
}
376380
}
377381

378382
MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(re_sub_obj, 3, 5, re_sub_helper);

‎extmod/modusocket.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,7 @@ STATIC mp_obj_t socket_recv(mp_obj_t self_in, mp_obj_t len_in) {
296296
return mp_const_empty_bytes;
297297
}
298298
vstr.len = ret;
299-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
299+
return mp_obj_new_bytes_from_vstr(&vstr);
300300
}
301301
STATIC MP_DEFINE_CONST_FUN_OBJ_2(socket_recv_obj, socket_recv);
302302

@@ -347,7 +347,7 @@ STATIC mp_obj_t socket_recvfrom(mp_obj_t self_in, mp_obj_t len_in) {
347347
tuple[0] = mp_const_empty_bytes;
348348
} else {
349349
vstr.len = ret;
350-
tuple[0] = mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
350+
tuple[0] = mp_obj_new_bytes_from_vstr(&vstr);
351351
}
352352
tuple[1] = netutils_format_inet_addr(ip, port, NETUTILS_BIG);
353353
return mp_obj_new_tuple(2, tuple);

‎extmod/vfs.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,7 @@ mp_obj_t mp_vfs_getcwd(void) {
360360
if (!(cwd[0] == '/' && cwd[1] == 0)) {
361361
vstr_add_str(&vstr, cwd);
362362
}
363-
return mp_obj_new_str_from_vstr(&mp_type_str, &vstr);
363+
return mp_obj_new_str_from_vstr(&vstr);
364364
}
365365
MP_DEFINE_CONST_FUN_OBJ_0(mp_vfs_getcwd_obj, mp_vfs_getcwd);
366366

‎ports/cc3200/mods/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ STATIC mp_obj_t os_urandom(mp_obj_t num) {
116116
for (int i = 0; i < n; i++) {
117117
vstr.buf[i] = rng_get();
118118
}
119-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
119+
return mp_obj_new_bytes_from_vstr(&vstr);
120120
}
121121
STATIC MP_DEFINE_CONST_FUN_OBJ_1(os_urandom_obj, os_urandom);
122122

‎ports/cc3200/mods/modusocket.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -581,7 +581,7 @@ STATIC mp_obj_t socket_recv(mp_obj_t self_in, mp_obj_t len_in) {
581581
}
582582
vstr.len = ret;
583583
vstr.buf[vstr.len] = '\0';
584-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
584+
return mp_obj_new_bytes_from_vstr(&vstr);
585585
}
586586
STATIC MP_DEFINE_CONST_FUN_OBJ_2(socket_recv_obj, socket_recv);
587587

@@ -625,7 +625,7 @@ STATIC mp_obj_t socket_recvfrom(mp_obj_t self_in, mp_obj_t len_in) {
625625
} else {
626626
vstr.len = ret;
627627
vstr.buf[vstr.len] = '\0';
628-
tuple[0] = mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
628+
tuple[0] = mp_obj_new_bytes_from_vstr(&vstr);
629629
}
630630
tuple[1] = netutils_format_inet_addr(ip, port, NETUTILS_LITTLE);
631631
return mp_obj_new_tuple(2, tuple);

‎ports/cc3200/mods/pybi2c.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -392,7 +392,7 @@ STATIC mp_obj_t pyb_i2c_readfrom(size_t n_args, const mp_obj_t *pos_args, mp_map
392392
pyb_i2c_read_into(args, &vstr);
393393

394394
// return the received data
395-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
395+
return mp_obj_new_bytes_from_vstr(&vstr);
396396
}
397397
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_i2c_readfrom_obj, 3, pyb_i2c_readfrom);
398398

@@ -456,7 +456,7 @@ STATIC mp_obj_t pyb_i2c_readfrom_mem(size_t n_args, const mp_obj_t *pos_args, mp
456456

457457
vstr_t vstr;
458458
pyb_i2c_readmem_into (args, &vstr);
459-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
459+
return mp_obj_new_bytes_from_vstr(&vstr);
460460
}
461461
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_i2c_readfrom_mem_obj, 1, pyb_i2c_readfrom_mem);
462462

‎ports/cc3200/mods/pybspi.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -305,7 +305,7 @@ STATIC mp_obj_t pyb_spi_read(size_t n_args, const mp_obj_t *pos_args, mp_map_t *
305305
pybspi_transfer(self, NULL, vstr.buf, vstr.len, &write);
306306

307307
// return the received data
308-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
308+
return mp_obj_new_bytes_from_vstr(&vstr);
309309
}
310310
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_spi_read_obj, 1, pyb_spi_read);
311311

‎ports/esp32/modsocket.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -552,7 +552,7 @@ mp_obj_t _socket_recvfrom(mp_obj_t self_in, mp_obj_t len_in,
552552
}
553553

554554
vstr.len = ret;
555-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
555+
return mp_obj_new_bytes_from_vstr(&vstr);
556556
}
557557

558558
STATIC mp_obj_t socket_recv(mp_obj_t self_in, mp_obj_t len_in) {

‎ports/esp32/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ STATIC mp_obj_t mp_uos_urandom(mp_obj_t num) {
4545
vstr.buf[i] = r;
4646
r >>= 8;
4747
}
48-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
48+
return mp_obj_new_bytes_from_vstr(&vstr);
4949
}
5050
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mp_uos_urandom_obj, mp_uos_urandom);
5151

‎ports/esp8266/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ STATIC mp_obj_t mp_uos_urandom(mp_obj_t num) {
4848
for (int i = 0; i < n; i++) {
4949
vstr.buf[i] = *WDEV_HWRNG;
5050
}
51-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
51+
return mp_obj_new_bytes_from_vstr(&vstr);
5252
}
5353
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mp_uos_urandom_obj, mp_uos_urandom);
5454

‎ports/mimxrt/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ STATIC mp_obj_t mp_uos_urandom(mp_obj_t num) {
6363
trng_start();
6464
TRNG_GetRandomData(TRNG, vstr.buf, n);
6565

66-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
66+
return mp_obj_new_bytes_from_vstr(&vstr);
6767
}
6868
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mp_uos_urandom_obj, mp_uos_urandom);
6969
#endif

‎ports/nrf/modules/machine/spi.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -388,7 +388,7 @@ STATIC mp_obj_t mp_machine_spi_read(size_t n_args, const mp_obj_t *args) {
388388
vstr_init_len(&vstr, mp_obj_get_int(args[1]));
389389
memset(vstr.buf, n_args == 3 ? mp_obj_get_int(args[2]) : 0, vstr.len);
390390
spi_transfer(args[0], vstr.len, vstr.buf, vstr.buf);
391-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
391+
return mp_obj_new_bytes_from_vstr(&vstr);
392392
}
393393
MP_DEFINE_CONST_FUN_OBJ_VAR_BETWEEN(mp_machine_spi_read_obj, 2, 3, mp_machine_spi_read);
394394

‎ports/nrf/modules/uos/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ STATIC mp_obj_t os_urandom(mp_obj_t num) {
108108
for (int i = 0; i < n; i++) {
109109
vstr.buf[i] = (uint8_t)(rng_generate_random_word() & 0xFF);
110110
}
111-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
111+
return mp_obj_new_bytes_from_vstr(&vstr);
112112
}
113113
STATIC MP_DEFINE_CONST_FUN_OBJ_1(os_urandom_obj, os_urandom);
114114
#endif

‎ports/rp2/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,6 @@ STATIC mp_obj_t mp_uos_urandom(mp_obj_t num) {
3535
for (int i = 0; i < n; i++) {
3636
vstr.buf[i] = rosc_random_u8(8);
3737
}
38-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
38+
return mp_obj_new_bytes_from_vstr(&vstr);
3939
}
4040
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mp_uos_urandom_obj, mp_uos_urandom);

‎ports/stm32/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ STATIC mp_obj_t mp_uos_urandom(mp_obj_t num) {
5858
for (int i = 0; i < n; i++) {
5959
vstr.buf[i] = rng_get();
6060
}
61-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
61+
return mp_obj_new_bytes_from_vstr(&vstr);
6262
}
6363
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mp_uos_urandom_obj, mp_uos_urandom);
6464
#endif

‎ports/stm32/pyb_i2c.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -926,7 +926,7 @@ STATIC mp_obj_t pyb_i2c_recv(size_t n_args, const mp_obj_t *pos_args, mp_map_t *
926926
if (o_ret != MP_OBJ_NULL) {
927927
return o_ret;
928928
} else {
929-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
929+
return mp_obj_new_bytes_from_vstr(&vstr);
930930
}
931931
}
932932
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_i2c_recv_obj, 1, pyb_i2c_recv);
@@ -1002,7 +1002,7 @@ STATIC mp_obj_t pyb_i2c_mem_read(size_t n_args, const mp_obj_t *pos_args, mp_map
10021002
if (o_ret != MP_OBJ_NULL) {
10031003
return o_ret;
10041004
} else {
1005-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
1005+
return mp_obj_new_bytes_from_vstr(&vstr);
10061006
}
10071007
}
10081008
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_i2c_mem_read_obj, 1, pyb_i2c_mem_read);

‎ports/stm32/pyb_spi.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ STATIC mp_obj_t pyb_spi_recv(size_t n_args, const mp_obj_t *pos_args, mp_map_t *
229229
if (o_ret != MP_OBJ_NULL) {
230230
return o_ret;
231231
} else {
232-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
232+
return mp_obj_new_bytes_from_vstr(&vstr);
233233
}
234234
}
235235
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_spi_recv_obj, 1, pyb_spi_recv);
@@ -298,7 +298,7 @@ STATIC mp_obj_t pyb_spi_send_recv(size_t n_args, const mp_obj_t *pos_args, mp_ma
298298
if (o_ret != MP_OBJ_NULL) {
299299
return o_ret;
300300
} else {
301-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr_recv);
301+
return mp_obj_new_bytes_from_vstr(&vstr_recv);
302302
}
303303
}
304304
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_spi_send_recv_obj, 1, pyb_spi_send_recv);

‎ports/stm32/usb.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -813,7 +813,7 @@ STATIC mp_obj_t pyb_usb_vcp_recv(size_t n_args, const mp_obj_t *args, mp_map_t *
813813
return mp_obj_new_int(ret); // number of bytes read into given buffer
814814
} else {
815815
vstr.len = ret; // set actual number of bytes read
816-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr); // create a new buffer
816+
return mp_obj_new_bytes_from_vstr(&vstr); // create a new buffer
817817
}
818818
}
819819
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_usb_vcp_recv_obj, 1, pyb_usb_vcp_recv);
@@ -1007,7 +1007,7 @@ STATIC mp_obj_t pyb_usb_hid_recv(size_t n_args, const mp_obj_t *args, mp_map_t *
10071007
return mp_obj_new_int(ret); // number of bytes read into given buffer
10081008
} else {
10091009
vstr.len = ret; // set actual number of bytes read
1010-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr); // create a new buffer
1010+
return mp_obj_new_bytes_from_vstr(&vstr); // create a new buffer
10111011
}
10121012
}
10131013
STATIC MP_DEFINE_CONST_FUN_OBJ_KW(pyb_usb_hid_recv_obj, 1, pyb_usb_hid_recv);

‎ports/unix/main.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -554,7 +554,7 @@ MP_NOINLINE int main_(int argc, char **argv) {
554554
vstr_init(&vstr, home_l + (p1 - p - 1) + 1);
555555
vstr_add_strn(&vstr, home, home_l);
556556
vstr_add_strn(&vstr, p + 1, p1 - p - 1);
557-
path_items[i] = mp_obj_new_str_from_vstr(&mp_type_str, &vstr);
557+
path_items[i] = mp_obj_new_str_from_vstr(&vstr);
558558
} else {
559559
path_items[i] = mp_obj_new_str_via_qstr(p, p1 - p);
560560
}
@@ -650,7 +650,7 @@ MP_NOINLINE int main_(int argc, char **argv) {
650650
vstr_init(&vstr, len + sizeof(".__main__"));
651651
vstr_add_strn(&vstr, argv[a + 1], len);
652652
vstr_add_strn(&vstr, ".__main__", sizeof(".__main__") - 1);
653-
import_args[0] = mp_obj_new_str_from_vstr(&mp_type_str, &vstr);
653+
import_args[0] = mp_obj_new_str_from_vstr(&vstr);
654654
goto reimport;
655655
}
656656

‎ports/unix/moduos.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ STATIC mp_obj_t mp_uos_urandom(mp_obj_t num) {
9494
vstr_t vstr;
9595
vstr_init_len(&vstr, n);
9696
mp_hal_get_random(n, vstr.buf);
97-
return mp_obj_new_str_from_vstr(&mp_type_bytes, &vstr);
97+
return mp_obj_new_bytes_from_vstr(&vstr);
9898
}
9999
STATIC MP_DEFINE_CONST_FUN_OBJ_1(mp_uos_urandom_obj, mp_uos_urandom);
100100

‎ports/unix/modusocket.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -559,7 +559,7 @@ STATIC mp_obj_t mod_socket_inet_ntop(mp_obj_t family_in, mp_obj_t binaddr_in) {
559559
mp_raise_OSError(errno);
560560
}
561561
vstr.len = strlen(vstr.buf);
562-
return mp_obj_new_str_from_vstr(&mp_type_str, &vstr);
562+
return mp_obj_new_str_from_vstr(&vstr);
563563
}
564564
STATIC MP_DEFINE_CONST_FUN_OBJ_2(mod_socket_inet_ntop_obj, mod_socket_inet_ntop);
565565

0 commit comments

Comments
 (0)