D24085: Cleanup and modernize the Calendar library codebase
Daniel Vrátil
noreply at phabricator.kde.org
Thu Sep 19 16:12:47 BST 2019
dvratil created this revision.
dvratil added a reviewer: barchiesi.
Herald added a project: KDE PIM.
Herald added a subscriber: kde-pim.
dvratil requested review of this revision.
REVISION SUMMARY
- enable C++14
- use QScopedPointer for d-pointers
- remove dead/unused code
- refactor duplicated code into functions
- use named constants instead of string literals to avoid typos but also to reduce the binary size due to string duplication
- const'ify and auto'ify
TEST PLAN
Compiles, tests pass
REPOSITORY
R477 KGAPI Library
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D24085
AFFECTED FILES
CMakeLists.txt
src/calendar/calendar.cpp
src/calendar/calendar.h
src/calendar/calendarcreatejob.cpp
src/calendar/calendarcreatejob.h
src/calendar/calendardeletejob.cpp
src/calendar/calendardeletejob.h
src/calendar/calendarfetchjob.cpp
src/calendar/calendarfetchjob.h
src/calendar/calendarmodifyjob.cpp
src/calendar/calendarmodifyjob.h
src/calendar/calendarservice.cpp
src/calendar/calendarservice.h
src/calendar/event.cpp
src/calendar/event.h
src/calendar/eventcreatejob.cpp
src/calendar/eventcreatejob.h
src/calendar/eventdeletejob.cpp
src/calendar/eventdeletejob.h
src/calendar/eventfetchjob.cpp
src/calendar/eventfetchjob.h
src/calendar/eventmodifyjob.cpp
src/calendar/eventmodifyjob.h
src/calendar/eventmovejob.cpp
src/calendar/eventmovejob.h
src/calendar/freebusyqueryjob.cpp
src/calendar/freebusyqueryjob.h
src/calendar/reminder.cpp
src/calendar/reminder.h
To: dvratil, barchiesi
Cc: kde-pim, fbampaloukas, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190919/325685eb/attachment.html>
More information about the kde-pim
mailing list