Require Qt 5.6: We align with what we're really testing We can use Q_ENUM/Q_FLAG instead of Q_ENUMS/Q_FLAGS (Qt 5.5) CursorImageInfo.cpp Join two QString::arg into one SessionsModel.cpp UsersModel.cpp Remove useless Q_D dashconnection.cpp ColorTheme.cpp UserMetrics.cpp horizontaljournaltest.cpp listviewwithpageheadersectionexternalmodeltest.cpp organicgridtest.cpp verticaljournaltest.cpp QObjects should have the Q_OBJECT macro indicatorsmodel.h fakeindicatorsmodel.h There's no Roles enum, remove appdrawerproxymodel.cpp MirSurfaceItem.cpp Add & to the foreach variable appdrawerproxymodel.cpp Make the string comparison faster easingcurve.h Remove QEasingCurve::Type declaration, it's already in Qt unitysortfilterproxymodelqml.h fake_scopesoverview.h Make some parent functions accessible to remove clang warnings PageList.cpp unitymenumodel.cpp qinputdeviceinfo_mock.cpp Better way to iterate the container Greeter.cpp Cheaper env var checking mediaplayer.cpp mediaplayer.h Rename timerEvent, clang was not happy since QObject has another one ApplicationInfo.h Mark as override MockAppDrawerModel.cpp launchermodeltest.cpp screens.cpp Fix warning fake_previewwidgetmodel.h Forward declare struct the same way it is defined later fake_scopes.cpp Safer connect horizontaljournaltest.cpp verticaljournaltest.cpp testutil.cpp Use at instead of [] in temporary launchermodelastest.cpp Remove unused member Use toModelIndex() instead of value<QModelIndex>() Approved by: Michael Zanetti, Unity8 CI Bot