* Make all MoveHint code value-based instead of pointer-based. MoveHint
   objects aren't that big. I don't think we gain much by passing around
   pointers and using values means we don't have to worry about cleanup.
 * Add a default contructor and an isValid() method to MoveHint. These
   replace the use of null pointers to represent invalid MoveHints.
 * Add an operator< to MoveHint to allow sorting by priority.
 * Remove DealerScenePrivate::hints as it made the hint code was overly
   complex. One always had to be mindful of the contents of d->hints and
   and be sure to clear it before adding to it. By making getHints()
   actually return the list of hints rather than modify d->hints, things
   get much simpler. As a result, we can get rid of hints(), newHint()
   and clearHints().
 * Fix an bug in how the next move is selected from the list of hints if
   we don't have a solution to the current game.

svn path=/trunk/KDE/kdegames/kpat/; revision=1132424