Shorten the freecell solutions.
Needs ReviewPublic

Authored by shlomif on Apr 8 2019, 6:13 PM.

Details

Reviewers
None
Group Reviewers
KDE Games
Summary

Based on new libfreecell-solver fixes and features.

Test Plan

Test that the freecell demo mode works with libfcs 5.2.0 or 5.6.0.

Diff Detail

Repository
R410 KPatience
Branch
shorten-freecell-solutions
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10626
Build 10644: arc lint + arc unit
shlomif created this revision.Apr 8 2019, 6:13 PM
Restricted Application added a reviewer: KDE Games. · View Herald TranscriptApr 8 2019, 6:13 PM
Restricted Application added a subscriber: kde-games-devel. · View Herald Transcript
shlomif requested review of this revision.Apr 8 2019, 6:13 PM
aacid added a subscriber: aacid.Apr 14 2019, 4:26 PM
aacid added inline comments.
fcs_soft_suspend_test.c
5

The expat license is not really compatible with the kde licensing policy, i guess you can change this snippet to something else?

https://community.kde.org/Policies/Licensing_Policy

@aacid : hi, the Expat License is a different (and more accurate) name to what that page calls MIT license. See https://en.wikipedia.org/wiki/MIT_License .

@aacid : hi, can we land this?