push changes from MessageViewer to kdepim (Viewer::UpdateMode -> UpdateMode).
ClosedPublic

Authored by knauss on Jan 29 2016, 6:22 PM.

Diff Detail

Repository
R43 KDE PIM
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
knauss updated this revision to Diff 2143.Jan 29 2016, 6:22 PM
knauss retitled this revision from to push changes from MessageViewer to kdepim (Viewer::UpdateMode -> UpdateMode)..
knauss updated this object.
knauss edited the test plan for this revision. (Show Details)
knauss added a reviewer: mlaurent.
Restricted Application added a project: KDE PIM. · View Herald TranscriptJan 29 2016, 6:22 PM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
mlaurent added inline comments.Jan 31 2016, 5:56 PM
plugins/messageviewer/bodypartformatter/calendar/text_calendar.cpp
235–236

SLOT ?

knauss updated this revision to Diff 2157.Jan 31 2016, 7:03 PM

change SLOT->SIGNAL

knauss added inline comments.Jan 31 2016, 7:08 PM
plugins/messageviewer/bodypartformatter/calendar/text_calendar.cpp
235–237

is there any way to use the new SIGNAL-SLOT syntax here?

asyncResultObserver is only a QObject Pointer, so it refuses to

knauss updated this revision to Diff 2162.Feb 1 2016, 3:39 PM

Using now enums.h

mlaurent edited edge metadata.Feb 1 2016, 9:16 PM

I don't see diff where you add enum.h in messageviewer ?
Which is the phabricator review ?

plugins/messageviewer/bodypartformatter/calendar/text_calendar.cpp
235–237

it's not possible indeed to use new connect api here

knauss marked an inline comment as done.Feb 2 2016, 2:01 AM

I don't see diff where you add enum.h in messageviewer ?
Which is the phabricator review ?

phab is a litle bit stupid. it is a already accepted request: D871

mlaurent accepted this revision.Feb 3 2016, 5:51 AM
mlaurent edited edge metadata.

Seems ok ship it

This revision is now accepted and ready to land.Feb 3 2016, 5:51 AM
This revision was automatically updated to reflect the committed changes.