djsundog/circuitpython
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge remote-tracking branch 'origin/main'

Browse Source
This commit is contained in:
Hosted Weblate 2023-03-28 16:30:15 +02:00
commit 618cfb0dc3
No known key found for this signature in database
GPG Key ID: A3FAAA06E6569B4C
3 changed files with 20 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

56
shared-bindings/displayio/Bitmap.c
View File

@ -62,14 +62,11 @@
//| ... //| ...
STATIC mp_obj_t displayio_bitmap_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *all_args) { STATIC mp_obj_t displayio_bitmap_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *all_args) {
mp_arg_check_num(n_args, n_kw, 3, 3, false); mp_arg_check_num(n_args, n_kw, 3, 3, false);
uint32_t width = mp_obj_get_int(all_args[0]); uint32_t width = mp_arg_validate_int_range(mp_obj_get_int(all_args[0]), 1, 32767, MP_QSTR_width);
uint32_t height = mp_obj_get_int(all_args[1]); uint32_t height = mp_arg_validate_int_range(mp_obj_get_int(all_args[1]), 1, 32767, MP_QSTR_height);
uint32_t value_count = mp_obj_get_int(all_args[2]); uint32_t value_count = mp_arg_validate_int_range(mp_obj_get_int(all_args[2]), 1, 65535, MP_QSTR_value_count);
uint32_t bits = 1; uint32_t bits = 1;
if (value_count == 0) {
mp_raise_ValueError(translate("value_count must be > 0"));
}
while ((value_count - 1) >> bits) { while ((value_count - 1) >> bits) {
if (bits < 8) { if (bits < 8) {
bits <<= 1; bits <<= 1;
@ -154,28 +151,24 @@ STATIC mp_obj_t bitmap_subscr(mp_obj_t self_in, mp_obj_t index_obj, mp_obj_t val
uint16_t x = 0; uint16_t x = 0;
uint16_t y = 0; uint16_t y = 0;
if (mp_obj_is_small_int(index_obj)) { if (mp_obj_is_small_int(index_obj)) {
mp_int_t i = MP_OBJ_SMALL_INT_VALUE(index_obj); mp_int_t i = mp_arg_validate_int_min(MP_OBJ_SMALL_INT_VALUE(index_obj), 0, MP_QSTR_index);
uint16_t width = common_hal_displayio_bitmap_get_width(self); uint16_t width = common_hal_displayio_bitmap_get_width(self);
x = i % width; x = i % width;
y = i / width; y = i / width;
} else { } else {
mp_obj_t *items; mp_obj_t *items;
mp_obj_get_array_fixed_n(index_obj, 2, &items); mp_obj_get_array_fixed_n(index_obj, 2, &items);
x = mp_obj_get_int(items[0]); x = mp_arg_validate_int_range(mp_obj_get_int(items[0]), 0, self->width - 1, MP_QSTR_x);
y = mp_obj_get_int(items[1]); y = mp_arg_validate_int_range(mp_obj_get_int(items[1]), 0, self->height - 1, MP_QSTR_y);
if (x >= common_hal_displayio_bitmap_get_width(self) || y >= common_hal_displayio_bitmap_get_height(self)) {
mp_raise_IndexError(translate("pixel coordinates out of bounds"));
}
} }
if (value_obj == MP_OBJ_SENTINEL) { if (value_obj == MP_OBJ_SENTINEL) {
// load // load
return MP_OBJ_NEW_SMALL_INT(common_hal_displayio_bitmap_get_pixel(self, x, y)); return MP_OBJ_NEW_SMALL_INT(common_hal_displayio_bitmap_get_pixel(self, x, y));
} else { } else {
mp_uint_t value = (mp_uint_t)mp_obj_get_int(value_obj); mp_uint_t value = (mp_uint_t)mp_arg_validate_int_range(
if ((value >> common_hal_displayio_bitmap_get_bits_per_value(self)) != 0) { mp_obj_get_int(value_obj), 0,
mp_raise_ValueError(translate("pixel value requires too many bits")); (UINT32_MAX >> (32 - common_hal_displayio_bitmap_get_bits_per_value(self))), MP_QSTR_value);
}
common_hal_displayio_bitmap_set_pixel(self, x, y, value); common_hal_displayio_bitmap_set_pixel(self, x, y, value);
} }
return mp_const_none; return mp_const_none;
@ -226,8 +219,9 @@ STATIC mp_obj_t displayio_bitmap_obj_blit(size_t n_args, const mp_obj_t *pos_arg
displayio_bitmap_t *self = MP_OBJ_TO_PTR(pos_args[0]); displayio_bitmap_t *self = MP_OBJ_TO_PTR(pos_args[0]);
check_for_deinit(self); check_for_deinit(self);
int16_t x = args[ARG_x].u_int; // Check x,y are within self (target) bitmap boundary
int16_t y = args[ARG_y].u_int; int16_t x = mp_arg_validate_int_range(args[ARG_x].u_int, 0, self->width - 1, MP_QSTR_x);
int16_t y = mp_arg_validate_int_range(args[ARG_y].u_int, 0, self->height - 1, MP_QSTR_y);
displayio_bitmap_t *source = mp_arg_validate_type(args[ARG_source].u_obj, &displayio_bitmap_type, MP_QSTR_source_bitmap); displayio_bitmap_t *source = mp_arg_validate_type(args[ARG_source].u_obj, &displayio_bitmap_type, MP_QSTR_source_bitmap);
@ -237,32 +231,21 @@ STATIC mp_obj_t displayio_bitmap_obj_blit(size_t n_args, const mp_obj_t *pos_arg
mp_raise_ValueError(translate("source palette too large")); mp_raise_ValueError(translate("source palette too large"));
} }
int16_t x1 = args[ARG_x1].u_int; // Check x1,y1,x2,y2 are within source bitmap boundary
int16_t y1 = args[ARG_y1].u_int; int16_t x1 = mp_arg_validate_int_range(args[ARG_x1].u_int, 0, source->width - 1, MP_QSTR_x1);
int16_t y1 = mp_arg_validate_int_range(args[ARG_y1].u_int, 0, source->height - 1, MP_QSTR_y1);
int16_t x2, y2; int16_t x2, y2;
// if x2 or y2 is None, then set as the maximum size of the source bitmap // if x2 or y2 is None, then set as the maximum size of the source bitmap
if (args[ARG_x2].u_obj == mp_const_none) { if (args[ARG_x2].u_obj == mp_const_none) {
x2 = source->width; x2 = source->width;
} else { } else {
x2 = mp_obj_get_int(args[ARG_x2].u_obj); x2 = mp_arg_validate_int_range(mp_obj_get_int(args[ARG_x2].u_obj), 0, source->width, MP_QSTR_x2);
} }
// int16_t y2; // int16_t y2;
if (args[ARG_y2].u_obj == mp_const_none) { if (args[ARG_y2].u_obj == mp_const_none) {
y2 = source->height; y2 = source->height;
} else { } else {
y2 = mp_obj_get_int(args[ARG_y2].u_obj); y2 = mp_arg_validate_int_range(mp_obj_get_int(args[ARG_y2].u_obj), 0, source->height, MP_QSTR_y2);
}
// Check x,y are within self (target) bitmap boundary
if ((x < 0) || (y < 0) || (x > self->width) || (y > self->height)) {
mp_raise_ValueError(translate("out of range of target"));
}
// Check x1,y1,x2,y2 are within source bitmap boundary
if ((x1 < 0) || (x1 > source->width) ||
(y1 < 0) || (y1 > source->height) ||
(x2 < 0) || (x2 > source->width) ||
(y2 < 0) || (y2 > source->height)) {
mp_raise_ValueError(translate("out of range of source"));
} }
// Ensure x1 < x2 and y1 < y2 // Ensure x1 < x2 and y1 < y2
@ -301,10 +284,7 @@ STATIC mp_obj_t displayio_bitmap_obj_fill(mp_obj_t self_in, mp_obj_t value_obj)
displayio_bitmap_t *self = MP_OBJ_TO_PTR(self_in); displayio_bitmap_t *self = MP_OBJ_TO_PTR(self_in);
check_for_deinit(self); check_for_deinit(self);
mp_uint_t value = (mp_uint_t)mp_obj_get_int(value_obj); mp_uint_t value = (mp_uint_t)mp_arg_validate_int_range(mp_obj_get_int(value_obj), 0,(1u << common_hal_displayio_bitmap_get_bits_per_value(self)) - 1,MP_QSTR_value);
if ((value >> common_hal_displayio_bitmap_get_bits_per_value(self)) != 0) {
mp_raise_ValueError(translate("pixel value requires too many bits"));
}
common_hal_displayio_bitmap_fill(self, value); common_hal_displayio_bitmap_fill(self, value);
return mp_const_none; return mp_const_none;

2
shared-bindings/displayio/Group.c
View File

@ -56,7 +56,7 @@ STATIC mp_obj_t displayio_group_make_new(const mp_obj_type_t *type, size_t n_arg
mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)]; mp_arg_val_t args[MP_ARRAY_SIZE(allowed_args)];
mp_arg_parse_all_kw_array(n_args, n_kw, all_args, MP_ARRAY_SIZE(allowed_args), allowed_args, args); mp_arg_parse_all_kw_array(n_args, n_kw, all_args, MP_ARRAY_SIZE(allowed_args), allowed_args, args);
mp_int_t scale = mp_arg_validate_int_min(args[ARG_scale].u_int, 1, MP_QSTR_scale); mp_int_t scale = mp_arg_validate_int_range(args[ARG_scale].u_int, 1, 32767,MP_QSTR_scale);
displayio_group_t *self = m_new_obj(displayio_group_t); displayio_group_t *self = m_new_obj(displayio_group_t);
self->base.type = &displayio_group_type; self->base.type = &displayio_group_type;

2
shared-bindings/displayio/Palette.c
View File

@ -61,7 +61,7 @@ STATIC mp_obj_t displayio_palette_make_new(const mp_obj_type_t *type, size_t n_a
displayio_palette_t *self = m_new_obj(displayio_palette_t); displayio_palette_t *self = m_new_obj(displayio_palette_t);
self->base.type = &displayio_palette_type; self->base.type = &displayio_palette_type;
common_hal_displayio_palette_construct(self, args[ARG_color_count].u_int, args[ARG_dither].u_bool); common_hal_displayio_palette_construct(self, mp_arg_validate_int_range(args[ARG_color_count].u_int, 1, 32767, MP_QSTR_color_count), args[ARG_dither].u_bool);
return MP_OBJ_FROM_PTR(self); return MP_OBJ_FROM_PTR(self);
} }