Revision 1218296
Go back to digest for 6th February 2011Other in Games
Parker Coates committed changes in /trunk/KDE/kdegames/kpat:
Move the custom multiple step move logic from Freecell to DealerScene.
* Coolo wants to add similar functionality to Forty & Eight, so there
was a desire to abstract the existing Freecell code.
* Moving the code into DealerScene allowed for simpler implementations
in a couple areas.
* While the algorithm used should be unchanged, the code has been
almost completely rewritten and in many cases simplified. The new
implementation is probably less efficient than the previous one
(mostly do to more QList copying) but I think the improved
readability and maintainability are worth it.
* The code now has approximately 1000000% more comments.
File Changes
Modified 4 files
- /trunk/KDE/kdegames/kpat
- /dealer.cpp
- /dealer.h
- /freecell.cpp
- /freecell.h
4 files changed in total