This attempt amounts to a search/replace of most dbo* statements.
It relies on the fact that qCDebug() supports a printf() style invocation/syntax since Qt 5.3.
This change partially resolves Phabricator/Maniphest issue T5980.
Details
Details
- Reviewers
- None
- Group Reviewers
KDE Games
It compiles.
Diff Detail
Diff Detail
- Repository
- R417 KSudoku
- Branch
- pretty-print-operator-overload
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
This is mainly a first attempt to get the ball rolling. It kills off nearly all of the previously remaining dbo* statements with one fell swoop of Search & Replace.
Comment Actions
So basically the scope of the patch are all the "non controversial" statements.
qC* adds a newline, so the final \n should be removed. But then, change required anyway, at least the statements without arguments could be modified to use the other format. As you prefer.
Comment Actions
Some clean up work to remove QtDebug include from sudokuboard.h
Move contents of SudokuBoard::prettyPrint/SudokuBoard::print to BoardContentsPrettyPrinter::print().