Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#5356 - Sokoban patch: Unified duplicate code into a function

Attached to Project: Rockbox
Opened by Malcolm Tyrrell (Malcohol) - Saturday, 13 May 2006, 17:41 GMT+2
Task Type Patches
Category Games
Status Closed
Assigned To No-one
Player type All players
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

A sokoban patch: I unified four virtually identical blocks of code into a function. This has no effect on functionality, but it might make the code easier to maintain.

(Aside: Are patches like this too trivial to submit? I'm new to the project, so please tell me if they are :-)
   sokoban.patch (16.8 KiB)
This task depends upon

Closed by  Dominik Riebeling (bluebrother)
Tuesday, 13 February 2007, 11:13 GMT+2
Reason for closing:  Out of Date
Additional comments about closing:  closed upon request as superseded by  FS#6625  according to reporter.
Comment by Malcolm Tyrrell (Malcohol) - Saturday, 13 May 2006, 17:53 GMT+2
...in fact, after the above change, I can remove some unnecessary local variables. This new patch replaces the earlier one.
   sokoban.patch (17 KiB)
Comment by Malcolm Tyrrell (Malcohol) - Saturday, 13 May 2006, 20:45 GMT+2
Argh! My first patch was dodgy (iPods don't define one of button defs I used). Here's yet another patch. Sorry!
   sokoban.patch (17 KiB)
Comment by Malcolm Tyrrell (Malcohol) - Tuesday, 13 February 2007, 10:57 GMT+2

This task can now be closed. It is superseded by  FS#6625  was has
been committed.

Loading...