This change resolves Phabricator/Maniphest task T5977.
Details
Details
- Reviewers
ltoscano - Group Reviewers
KDE Games - Maniphest Tasks
- T5977: Sudoku: use QFileInfoList in KSudoku::updateShapesList()
- Commits
- R417:52bdce6a0d93: Cleanup KSudoku::updateShapesList() to remove manual path fiddling.
Compiled & tested I could still generate custom game variant puzzles: Mathdoku 4x4, Killer 9x9.
Also, to confirm the duplicate filtering is redundant I ran ksudoku with a deliberately messed up XDG_DATA_DIRS environment variable:
XDG_DATA_DIRS="$XDG_DATA_DIRS:$PWD/kde-5/share;$PWD/kde-5/share;$PWD/kde-5/share" ./kde-5/bin/ksudoku
(Which still lists each game variant type only once, meaning the filtering was in fact redundant.)
Diff Detail
Diff Detail
- Repository
- R417 KSudoku
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
I think it's safe to skip the contains() check entirely, and I'm tempted to remove it. Did I overlook something?