Fix missing includes
ClosedPublic

Authored by carewolf on Mar 7 2018, 12:21 PM.

Details

Summary

Don't rely other headers to indirectly include what we need.

Diff Detail

Repository
R206 KMail
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
carewolf created this revision.Mar 7 2018, 12:21 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptMar 7 2018, 12:21 PM
carewolf requested review of this revision.Mar 7 2018, 12:21 PM
mlaurent accepted this revision.Mar 7 2018, 1:06 PM
This revision is now accepted and ready to land.Mar 7 2018, 1:06 PM
This revision was automatically updated to reflect the committed changes.
cfeck added a subscriber: cfeck.Mar 7 2018, 4:48 PM

According to our release schedules, the 'Applications/17.12' branches are already frozen for tagging. Are the recent KDEPIM commits worth a respin of the tarballs? Note that there are several applications that fail to compile with Qt 5.11, but we agreed that we only fix these in 'master' branch for the upcoming KDE Applications 18.04 releases, because Qt 5.11 will only be released end of May.

No sorry, I just pushed them to stable. I wasn't aware it was in freeze. Since this is for Qt 5.11 support it is not urgent.