shithub: puzzles

Info  •  Files  •  Log  •  Branches

Don't create an undo-chain entry for a move with no effect.

[originally from svn r7333]

--- a/filling.c

+++ b/filling.c

@@ -962,9 +962,13 @@

     {

         const int i = w*ui->y + ui->x;

         char buf[64];

-        sprintf(buf, "%d_%d", i, button);

         ui->x = ui->y = -1;

-        return dupstr(buf);

+	if (state->board[i] == button) {

+	    return "";		       /* no change - just update ui */

+	} else {

+	    sprintf(buf, "%d_%d", i, button);

+	    return dupstr(buf);

+	}

     }

 }