Skip to content

py/map: use MP_ERROR_TEXT for mp_map_presize error message#25

Merged
romanz merged 1 commit intotrezor-v1.19.1from
romanz/fix-presize-error
May 14, 2025
Merged

py/map: use MP_ERROR_TEXT for mp_map_presize error message#25
romanz merged 1 commit intotrezor-v1.19.1from
romanz/fix-presize-error

Conversation

@romanz
Copy link
Copy Markdown

@romanz romanz commented May 13, 2025

Similar to other call sites of mp_raise_msg_varg:

py/runtime.c
951:    mp_raise_msg_varg(&mp_type_ValueError, MP_ERROR_TEXT("need more than %d values to unpack"), (int)seq_len);
957:    mp_raise_msg_varg(&mp_type_ValueError, MP_ERROR_TEXT("too many values to unpack (expected %d)"), (int)num);
1018:    mp_raise_msg_varg(&mp_type_ValueError, MP_ERROR_TEXT("need more than %d values to unpack"), (int)seq_len);

Following https://github.com/trezor/trezor-firmware/pull/4961/files#r2086986643.

@romanz romanz merged commit 5e38f62 into trezor-v1.19.1 May 14, 2025
15 of 74 checks passed
@romanz romanz deleted the romanz/fix-presize-error branch May 14, 2025 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants