diff options
| author | Jeff Epler <jepler@gmail.com> | 2020-04-13 08:51:35 -0500 |
|---|---|---|
| committer | Jeff Epler <jepler@gmail.com> | 2020-04-14 18:24:59 -0500 |
| commit | 880fff80e904ef4b173d551742b7acde56114f0c (patch) | |
| tree | 9942c0427cc31bb08f299123e4abb80416b5e7f2 /shared-bindings/framebufferio/FramebufferDisplay.c | |
| parent | 5d328c3b449dbd0eaf40c2edc205fb7c5d381589 (diff) | |
protomatter: Respond to review comments
- rename oe_pin -> output_enable_pin
- improve and reorganize docstrings
- rename swapbuffers->refresh
- rename "paused" -> "brightness", change semantics slightly
- common_hal several functions
- clarify why the common_hal routines can't be used directly in the
protocol's function pointers
- whitespace cleanups
- remove prototypes for nonexistent functions
Diffstat (limited to 'shared-bindings/framebufferio/FramebufferDisplay.c')
| -rw-r--r-- | shared-bindings/framebufferio/FramebufferDisplay.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/shared-bindings/framebufferio/FramebufferDisplay.c b/shared-bindings/framebufferio/FramebufferDisplay.c index 538d3afb4..aa8fc1a63 100644 --- a/shared-bindings/framebufferio/FramebufferDisplay.c +++ b/shared-bindings/framebufferio/FramebufferDisplay.c @@ -216,7 +216,7 @@ STATIC mp_obj_t framebufferio_framebufferdisplay_obj_set_brightness(mp_obj_t sel framebufferio_framebufferdisplay_obj_t *self = native_display(self_in); common_hal_framebufferio_framebufferdisplay_set_auto_brightness(self, false); mp_float_t brightness = mp_obj_get_float(brightness_obj); - if (brightness < 0 || brightness > 1.0) { + if (brightness < 0.0f || brightness > 1.0f) { mp_raise_ValueError(translate("Brightness must be 0-1.0")); } bool ok = common_hal_framebufferio_framebufferdisplay_set_brightness(self, brightness); |