pbio/sys/hmi_none: Check for shutdown at least once.

laurensvalk · laurensvalk · commit b1692d826378 · 2025-11-02T19:34:43.000+01:00
If shutdown was already requested before the HMI ran, exit right away.

Allows hub.system.shutdown() to work in the simulated hub.
diff --git a/lib/pbio/sys/hmi_none.c b/lib/pbio/sys/hmi_none.c
@@ -25,12 +25,12 @@ void pbsys_hmi_deinit(void) {
 
 pbio_error_t pbsys_hmi_await_program_selection(void) {
 
-    while (pbdrv_button_get_pressed()) {
+    do {
         if (pbsys_status_test(PBIO_PYBRICKS_STATUS_SHUTDOWN_REQUEST)) {
             return PBIO_ERROR_CANCELED;
         }
         pbio_os_run_processes_and_wait_for_event();
-    }
+    } while (pbdrv_button_get_pressed());
 
     return pbsys_main_program_request_start(PBIO_PYBRICKS_USER_PROGRAM_ID_REPL, PBSYS_MAIN_PROGRAM_START_REQUEST_TYPE_BOOT);
 }

Original file line number	Diff line number	Diff line change
`@@ -25,12 +25,12 @@ void pbsys_hmi_deinit(void) {`
`25`	`25`
`26`	`26`	`pbio_error_t pbsys_hmi_await_program_selection(void) {`
`27`	`27`
`28`		`- while (pbdrv_button_get_pressed()) {`
	`28`	`+ do {`
`29`	`29`	`if (pbsys_status_test(PBIO_PYBRICKS_STATUS_SHUTDOWN_REQUEST)) {`
`30`	`30`	`return PBIO_ERROR_CANCELED;`
`31`	`31`	`}`
`32`	`32`	`pbio_os_run_processes_and_wait_for_event();`
`33`		`- }`
	`33`	`+ } while (pbdrv_button_get_pressed());`
`34`	`34`
`35`	`35`	`return pbsys_main_program_request_start(PBIO_PYBRICKS_USER_PROGRAM_ID_REPL, PBSYS_MAIN_PROGRAM_START_REQUEST_TYPE_BOOT);`
`36`	`36`	`}`