shithub: puzzles

Download patch

ref: 205508e969cdb6aed54a1b75367e7bb3891c53ba
parent: 029cc3996845899edf1cfae55fbf3bd55cbc7439
author: Ben Harris <bjh21@bjh21.me.uk>
date: Wed Aug 9 20:23:09 EDT 2023

Mosaic: remove some unused structure members

As far as I could tell, cur_x, cur_y, prev_cur_x, and prev_cur_y in
game_drawstate were never used.  prev_cur_x and prev_cur_y in game_ui
were assigned but never referenced.  So they may as well all go.

--- a/mosaic.c
+++ b/mosaic.c
@@ -126,7 +126,6 @@
     bool in_progress;
     int last_x, last_y, last_state;
     int cur_x, cur_y;
-    int prev_cur_x, prev_cur_y;
     bool cur_visible;
 };
 
@@ -133,8 +132,6 @@
 struct game_drawstate {
     int tilesize;
     int *state;
-    int cur_x, cur_y;           /* -1, -1 for no cursor displayed. */
-    int prev_cur_x, prev_cur_y;
 };
 
 static game_params *default_params(void)
@@ -1187,8 +1184,6 @@
         }
         ui->cur_visible = false;
     } else if (IS_CURSOR_MOVE(button)) {
-        ui->prev_cur_x = ui->cur_x;
-        ui->prev_cur_y = ui->cur_y;
         move_cursor(button, &ui->cur_x, &ui->cur_y, state->width,
                     state->height, false, NULL);
         ui->cur_visible = true;