Revision 1218296

Go back to digest for 6th February 2011

Other 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