avoid flicker in debug frame display and format as hex for fixed width

src/game.c

@@ -262,7 +262,7 @@ int32_t game(enum GAME_MODE mode) NONBANKED {
         }
     }
 
-    uint8_t x_off = win_game_draw(score);
+    uint8_t x_off = win_game_draw(score, 1);
     move_win(MINWNDPOSX + DEVICE_SCREEN_PX_WIDTH - x_off, MINWNDPOSY + DEVICE_SCREEN_PX_HEIGHT - 16);
 
     SHOW_WIN;
@@ -497,7 +497,7 @@ int32_t game(enum GAME_MODE mode) NONBANKED {
         hide_sprites_range(hiwater, MAX_HARDWARE_SPRITES);
 
         if ((score != prev_score) || ((_cpu == CGB_TYPE) && (conf_get()->debug_flags))) {
-            uint8_t x_off = win_game_draw(score);
+            uint8_t x_off = win_game_draw(score, 0);
             move_win(MINWNDPOSX + DEVICE_SCREEN_PX_WIDTH - x_off, MINWNDPOSY + DEVICE_SCREEN_PX_HEIGHT - 16);
         }
 

src/strings.c

@@ -49,9 +49,9 @@ static const char string_error[] = "error";
 static const char string_gb_printer[] = "GB Printer";
 static const char string_score_printout[] = "Score Printout";
 static const char string_result[] = "Result:";
-static const char string_printf_error[] = "error: 0x%04x";
-static const char string_printf_frames[] = "Frames: %u";
-static const char string_printf_timer[] = "Timer: %u";
+static const char string_printf_error[] = "error: 0x%x";
+static const char string_printf_frames[] = "Frames: 0x%x";
+static const char string_printf_timer[] = " Timer: 0x%x";
 
 static const char * const strings[COUNT_STRINGS] = {
     string_top,            // STR_TOP

src/window.c

@@ -258,7 +258,7 @@ void win_name_draw(uint16_t name, uint8_t is_black, uint8_t pos) BANKED {
          (pos == 2) ? !is_black : is_black);
 }
 
-uint8_t win_game_draw(int32_t score) BANKED {
+uint8_t win_game_draw(int32_t score, uint8_t initial) BANKED {
     uint8_t is_black = 0;
     if (score < 0) {
         score = -score;
@@ -266,16 +266,21 @@ uint8_t win_game_draw(int32_t score) BANKED {
     }
 
     if ((_cpu == CGB_TYPE) && (conf_get()->debug_flags)) {
-        // TODO hard-coded black bg tile
-        fill_win(0, 0, 20, 2, 0x80, BKGF_CGB_PAL3);
+        static int32_t prev_score = 0;
+        if (initial || (score != prev_score)) {
+            prev_score = score;
+
+            // TODO hard-coded black bg tile
+            fill_win(0, 0, 20, 2, 0x80, BKGF_CGB_PAL3);
+        }
 
         uint8_t x_off = number(score, 0, 0, is_black) >> 3;
 
         sprintf(str_buff, get_string(STR_PRINTF_FRAMES), (uint16_t)game_get_framecount());
-        str_ascii(str_buff, x_off + 1, 0, 1);
+        str_ascii(str_buff, x_off, 0, 1);
 
         sprintf(str_buff, get_string(STR_PRINTF_TIMER), (uint16_t)timer_get());
-        str_ascii(str_buff, x_off + 1, 1, 1);
+        str_ascii(str_buff, x_off, 1, 1);
 
         return DEVICE_SCREEN_PX_WIDTH;
     } else {

src/window.h

@@ -37,7 +37,7 @@ void win_conf(void) BANKED;
 void win_debug(void) BANKED;
 void win_name(int32_t score) BANKED;
 void win_name_draw(uint16_t name, uint8_t is_black, uint8_t pos) BANKED;
-uint8_t win_game_draw(int32_t score) BANKED;
+uint8_t win_game_draw(int32_t score, uint8_t initial) BANKED;
 
 void win_str_center(const char *s, uint8_t y_off, uint8_t is_black);