- remove KReportUnit::ListOption (T4967), add Invalid type, better support descriptions and symbols
- mark default constants as extern to avoid copying, add DEFAULT_UNIT_TYPE
Details
Details
- Reviewers
piggz - Maniphest Tasks
- T4967: Fix KReportUnit::ListOption API
- Commits
- R14:3689fe8962b9: API: Clean up KReportUnit
R14:d4af3605e666: Clean up KReportUnit
Build KReport with tests enabled, run tests, run KReportExample, try the Unit combo for the page units.
Build Kexi with this KReport - no changes needed.
(switching units for page is still broken but this is not related to this very patch)
Diff Detail
Diff Detail
- Repository
- R14 KReport
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.