WIP: fix moveCount for Freecell
Needs ReviewPublic

Authored by fabiank on Feb 16 2020, 12:16 PM.

Details

Reviewers
aacid
shlomif
yurchor
piotrkakol
Group Reviewers
KDE Games
Summary

Instead of changing the logic in dealer.cpp as in D27336, only change the moveCount function for FreeCell.

Pro:

  • less invasive change, less likely to cause regressions in other places

Contra:

  • This might only pamper over an actual issue, which D27336 would fix.
Test Plan
  • Started a new Freecell game (without multiple aces), and checked that the count is sound
  • Loaded existing Freecell games, and verified that the count is shown as expected
  • Started Freecell game where the initial state had two aces; verified that after the auto move completed, the displayed count was 2 (and not 1)

Diff Detail

Repository
R410 KPatience
Lint
Lint Skipped
Unit
Unit Tests Skipped
fabiank created this revision.Feb 16 2020, 12:16 PM
Restricted Application added a reviewer: KDE Games. · View Herald TranscriptFeb 16 2020, 12:16 PM
Restricted Application added a subscriber: kde-games-devel. · View Herald Transcript
fabiank requested review of this revision.Feb 16 2020, 12:16 PM