Skip to content
Snippets Groups Projects
Commit fe3cc5bb authored by Damien George's avatar Damien George
Browse files

py/modstruct: Use more compact mp_raise_ValueError function.

Saves a few bytes of code size.
parent 3b5affa0
No related branches found
No related tags found
No related merge requests found
...@@ -114,7 +114,7 @@ STATIC mp_obj_t struct_calcsize(mp_obj_t fmt_in) { ...@@ -114,7 +114,7 @@ STATIC mp_obj_t struct_calcsize(mp_obj_t fmt_in) {
mp_uint_t align; mp_uint_t align;
size_t sz = mp_binary_get_size(fmt_type, *fmt, &align); size_t sz = mp_binary_get_size(fmt_type, *fmt, &align);
if (sz == 0) { if (sz == 0) {
nlr_raise(mp_obj_new_exception_msg(&mp_type_ValueError, "unsupported format")); mp_raise_ValueError("unsupported format");
} }
while (cnt--) { while (cnt--) {
// Apply alignment // Apply alignment
...@@ -149,7 +149,7 @@ STATIC mp_obj_t struct_unpack_from(size_t n_args, const mp_obj_t *args) { ...@@ -149,7 +149,7 @@ STATIC mp_obj_t struct_unpack_from(size_t n_args, const mp_obj_t *args) {
// negative offsets are relative to the end of the buffer // negative offsets are relative to the end of the buffer
offset = bufinfo.len + offset; offset = bufinfo.len + offset;
if (offset < 0) { if (offset < 0) {
nlr_raise(mp_obj_new_exception_msg(&mp_type_ValueError, "buffer too small")); mp_raise_ValueError("buffer too small");
} }
} }
p += offset; p += offset;
...@@ -164,7 +164,7 @@ STATIC mp_obj_t struct_unpack_from(size_t n_args, const mp_obj_t *args) { ...@@ -164,7 +164,7 @@ STATIC mp_obj_t struct_unpack_from(size_t n_args, const mp_obj_t *args) {
sz = get_fmt_num(&fmt); sz = get_fmt_num(&fmt);
} }
if (p + sz > end_p) { if (p + sz > end_p) {
nlr_raise(mp_obj_new_exception_msg(&mp_type_ValueError, "buffer too small")); mp_raise_ValueError("buffer too small");
} }
mp_obj_t item; mp_obj_t item;
if (*fmt == 's') { if (*fmt == 's') {
...@@ -197,7 +197,7 @@ STATIC void struct_pack_into_internal(mp_obj_t fmt_in, byte *p, byte* end_p, siz ...@@ -197,7 +197,7 @@ STATIC void struct_pack_into_internal(mp_obj_t fmt_in, byte *p, byte* end_p, siz
sz = get_fmt_num(&fmt); sz = get_fmt_num(&fmt);
} }
if (p + sz > end_p) { if (p + sz > end_p) {
nlr_raise(mp_obj_new_exception_msg(&mp_type_ValueError, "buffer too small")); mp_raise_ValueError("buffer too small");
} }
if (*fmt == 's') { if (*fmt == 's') {
...@@ -240,7 +240,7 @@ STATIC mp_obj_t struct_pack_into(size_t n_args, const mp_obj_t *args) { ...@@ -240,7 +240,7 @@ STATIC mp_obj_t struct_pack_into(size_t n_args, const mp_obj_t *args) {
// negative offsets are relative to the end of the buffer // negative offsets are relative to the end of the buffer
offset = (mp_int_t)bufinfo.len + offset; offset = (mp_int_t)bufinfo.len + offset;
if (offset < 0) { if (offset < 0) {
nlr_raise(mp_obj_new_exception_msg(&mp_type_ValueError, "buffer too small")); mp_raise_ValueError("buffer too small");
} }
} }
byte *p = (byte *)bufinfo.buf; byte *p = (byte *)bufinfo.buf;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment