pull request #1272 - unify moverock()

Pull request from argrath:  change a couple of places in moverock()
to use moverock_done() and return early rather than use break to fall
to the end.

Closes #1272
This commit is contained in:
PatR
2024-09-03 23:51:38 -07:00

View File

@@ -546,16 +546,17 @@ moverock(void)
Sokoban rules because on next step you could go Sokoban rules because on next step you could go
past it without pushing it to plug a pit or hole */ past it without pushing it to plug a pit or hole */
sokoban_guilt(); sokoban_guilt();
break;
} }
break; moverock_done(sx, sy);
return 0;
} }
if (could_move_onto_boulder(sx, sy)) { if (could_move_onto_boulder(sx, sy)) {
pline( pline(
"However, you can squeeze yourself into a small opening."); "However, you can squeeze yourself into a small opening.");
sokoban_guilt(); sokoban_guilt();
break; moverock_done(sx, sy);
return 0;
} else { } else {
moverock_done(sx, sy); moverock_done(sx, sy);
return -1; return -1;