Skip to content

Commit 7dc7030

Browse files
committed
codal_port/microbit_soundeffect: Remove ability for fx to be None.
Fixes issue #130. Signed-off-by: Damien George <[email protected]>
1 parent 7f9c32e commit 7dc7030

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

src/codal_port/microbit_soundeffect.c

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -264,18 +264,11 @@ STATIC void microbit_soundeffect_attr(mp_obj_t self_in, qstr attr, mp_obj_t *des
264264
if (dest[0] == MP_OBJ_NULL) {
265265
// Load attribute.
266266
unsigned int value = sound_expr_decode(self, soundeffect_attr->offset, soundeffect_attr->length);
267-
if (attr == MP_QSTR_fx && value == 0) {
268-
dest[0] = mp_const_none;
269-
} else {
270-
dest[0] = MP_OBJ_NEW_SMALL_INT(value);
271-
}
267+
dest[0] = MP_OBJ_NEW_SMALL_INT(value);
272268
} else if (dest[1] != MP_OBJ_NULL) {
273269
// Store attribute.
274270
if (self->is_mutable) {
275-
unsigned int value = 0;
276-
if (dest[1] != mp_const_none) {
277-
value = mp_obj_get_int(dest[1]);
278-
}
271+
unsigned int value = mp_obj_get_int(dest[1]);
279272
sound_expr_encode(self, soundeffect_attr->offset, soundeffect_attr->length, value);
280273
dest[0] = MP_OBJ_NULL; // Indicate store succeeded.
281274
}

0 commit comments

Comments
 (0)