rainbowio: No need for the ternary here

jepler · jepler · commit 9646d3f9631d · 2021-11-12T17:31:15.000-06:00
.. mp_obj_get_float handles the case where the arg is an int already.

This saves 12 bytes of code size on trinket_m0.
diff --git a/shared-bindings/rainbowio/__init__.c b/shared-bindings/rainbowio/__init__.c
@@ -37,7 +37,8 @@
 //|     ...
 //|
 STATIC mp_obj_t rainbowio_colorwheel(mp_obj_t n) {
-    return MP_OBJ_NEW_SMALL_INT(colorwheel(mp_obj_is_small_int(n) ? MP_OBJ_SMALL_INT_VALUE(n) : mp_obj_get_float(n)));
+    mp_float_t f = mp_obj_get_float(n);
+    return MP_OBJ_NEW_SMALL_INT(colorwheel(f));
 }
 STATIC MP_DEFINE_CONST_FUN_OBJ_1(rainbowio_colorwheel_obj, rainbowio_colorwheel);
 

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,8 @@`
`37`	`37`	`//\| ...`
`38`	`38`	`//\|`
`39`	`39`	`STATIC mp_obj_t rainbowio_colorwheel(mp_obj_t n) {`
`40`		`- return MP_OBJ_NEW_SMALL_INT(colorwheel(mp_obj_is_small_int(n) ? MP_OBJ_SMALL_INT_VALUE(n) : mp_obj_get_float(n)));`
	`40`	`+ mp_float_t f = mp_obj_get_float(n);`
	`41`	`+ return MP_OBJ_NEW_SMALL_INT(colorwheel(f));`
`41`	`42`	`}`
`42`	`43`	`STATIC MP_DEFINE_CONST_FUN_OBJ_1(rainbowio_colorwheel_obj, rainbowio_colorwheel);`
`43`	`44`