framebufferio: Set type to none when releasing

jepler · jepler · commit 6d19a0988882 · 2020-08-12T07:32:18.000-05:00
This avoids the message "Too many displays in use" when they are released
directly, rather than via release_displays().
diff --git a/shared-module/framebufferio/FramebufferDisplay.c b/shared-module/framebufferio/FramebufferDisplay.c
@@ -309,6 +309,7 @@ void release_framebufferdisplay(framebufferio_framebufferdisplay_obj_t* self) {
     common_hal_framebufferio_framebufferdisplay_set_auto_refresh(self, false);
     release_display_core(&self->core);
     self->framebuffer_protocol->deinit(self->framebuffer);
+    self->base.type = &mp_type_NoneType;
 }
 
 void reset_framebufferdisplay(framebufferio_framebufferdisplay_obj_t* self) {

Original file line number	Diff line number	Diff line change
`@@ -309,6 +309,7 @@ void release_framebufferdisplay(framebufferio_framebufferdisplay_obj_t* self) {`
`309`	`309`	`common_hal_framebufferio_framebufferdisplay_set_auto_refresh(self, false);`
`310`	`310`	`release_display_core(&self->core);`
`311`	`311`	`self->framebuffer_protocol->deinit(self->framebuffer);`
	`312`	`+ self->base.type = &mp_type_NoneType;`
`312`	`313`	`}`
`313`	`314`
`314`	`315`	`void reset_framebufferdisplay(framebufferio_framebufferdisplay_obj_t* self) {`