Clean up KReportUnit

Authored by staniek on Jul 17 2017, 4:01 PM.

Description

Clean up KReportUnit

Summary:

  • remove KReportUnit::ListOption (T4967), add Invalid type, better support descriptions and symbols
  • mark default constants as extern to avoid copying, add DEFAULT_UNIT_TYPE

Test Plan:
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)

Reviewers: piggz

Reviewed By: piggz

Tags: KReport

Maniphest Tasks: T4967

Differential Revision: https://phabricator.kde.org/D6756

Details

Committed
staniekJul 21 2017, 10:07 PM
Reviewer
piggz
Differential Revision
D6756: Clean up KReportUnit
Parents
R14:98e3e5987868: Port to the new KPropertyListData API
Branches
Unknown
Tags
Unknown