Review Request: Adding a chronometer to the fifteenPuzzle plasmoid

[Alex Raymond]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4870/
-----------------------------------------------------------

Review request for Plasma and Aaron Seigo.


Summary
-------

This patch implements a chronometer for solving the puzzle. It starts as soon as it is shuffled, and automatically ends when the puzzle is sorted. A Plasma::Dialog is shown with the time elapsed after sorting out.


Diffs
-----

  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteen.h 1158685 
  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteen.cpp 1158685 
  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteenPuzzle.h 1158685 
  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteenPuzzle.cpp 1158685 

Diff: http://reviewboard.kde.org/r/4870/diff


Testing
-------

Probably it would only be testable if the position patch is applied, since the initial positions on the current revision create an unsolvable puzzle.


Thanks,

Alex

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100803/ba89b6c6/attachment-0001.htm