esp32/esp32_rmt: Don't do unnecessary check for unsigned less than zero.

dpgeorge · dpgeorge · commit 6f5e02639da7 · 2020-10-06T18:08:33.000+11:00
diff --git a/ports/esp32/esp32_rmt.c b/ports/esp32/esp32_rmt.c
@@ -209,7 +209,7 @@ STATIC mp_obj_t esp32_rmt_write_pulses(size_t n_args, const mp_obj_t *pos_args,
     mp_obj_t pulses = args[1].u_obj;
     mp_uint_t start = args[2].u_int;
 
-    if (start < 0 || start > 1) {
+    if (start > 1) {
         mp_raise_ValueError(MP_ERROR_TEXT("start must be 0 or 1"));
     }
 

Original file line number	Diff line number	Diff line change
`@@ -209,7 +209,7 @@ STATIC mp_obj_t esp32_rmt_write_pulses(size_t n_args, const mp_obj_t *pos_args,`
`209`	`209`	`mp_obj_t pulses = args[1].u_obj;`
`210`	`210`	`mp_uint_t start = args[2].u_int;`
`211`	`211`
`212`		`- if (start < 0 \|\| start > 1) {`
	`212`	`+ if (start > 1) {`
`213`	`213`	`mp_raise_ValueError(MP_ERROR_TEXT("start must be 0 or 1"));`
`214`	`214`	`}`
`215`	`215`