René J.V. Bertin
2015-04-13 15:29:10 UTC
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123353/
-----------------------------------------------------------
Review request for KDE Software on Mac OS X, KDEPIM and Thorsten Staerk.
Repository: kdepim
Description
-------
A while ago I created a RR for ktimetracker on OS X (https://git.reviewboard.kde.org/r/119533/) which was either never committed (despite a "Ship It") or which was reverted at a later stage. I therefore re-submit the patch with updates.
I have now a better fix for the failing dynamic_cast in mainwindow.cpp . The cast itself continues to fail despite ensuring that `ktimetrackerpart` is exported. It appears not to be required on OS X, however.
The idle time detection code for OS X has not been changed.
Diffs
-----
ktimetracker/CMakeLists.txt bf7b0ef
ktimetracker/idletimedetector.h 524f739
ktimetracker/idletimedetector.cpp 250dda5
ktimetracker/ktimetrackerpart.h 95da43b
ktimetracker/ktimetrackerpart_export.h PRE-CREATION
ktimetracker/ktimetrackerutility.cpp efeba3a
ktimetracker/mainwindow.cpp 56d4c28
Diff: https://git.reviewboard.kde.org/r/123353/diff/
Testing
-------
On OS X 10.9.5 with kdelibs 4.14.5
Thanks,
René J.V. Bertin
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123353/
-----------------------------------------------------------
Review request for KDE Software on Mac OS X, KDEPIM and Thorsten Staerk.
Repository: kdepim
Description
-------
A while ago I created a RR for ktimetracker on OS X (https://git.reviewboard.kde.org/r/119533/) which was either never committed (despite a "Ship It") or which was reverted at a later stage. I therefore re-submit the patch with updates.
I have now a better fix for the failing dynamic_cast in mainwindow.cpp . The cast itself continues to fail despite ensuring that `ktimetrackerpart` is exported. It appears not to be required on OS X, however.
The idle time detection code for OS X has not been changed.
Diffs
-----
ktimetracker/CMakeLists.txt bf7b0ef
ktimetracker/idletimedetector.h 524f739
ktimetracker/idletimedetector.cpp 250dda5
ktimetracker/ktimetrackerpart.h 95da43b
ktimetracker/ktimetrackerpart_export.h PRE-CREATION
ktimetracker/ktimetrackerutility.cpp efeba3a
ktimetracker/mainwindow.cpp 56d4c28
Diff: https://git.reviewboard.kde.org/r/123353/diff/
Testing
-------
On OS X 10.9.5 with kdelibs 4.14.5
Thanks,
René J.V. Bertin