mlaurent created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY Remove extra ';' Use override everywhere as we already use it in some part of code Fix potential crash when we delete a pointer TEST PLAN build/using :) REPOSITORY R119 Plasma Desktop BRANCH modernize_folderplugin REVISION DETAIL https://phabricator.kde.org/D8473 AFFECTED FILES containments/desktop/plugins/folder/directorypicker.cpp containments/desktop/plugins/folder/foldermodel.cpp containments/desktop/plugins/folder/foldermodel.h containments/desktop/plugins/folder/labelgenerator.cpp containments/desktop/plugins/folder/mimetypesmodel.cpp containments/desktop/plugins/folder/mimetypesmodel.h containments/desktop/plugins/folder/placesmodel.h containments/desktop/plugins/folder/positioner.cpp containments/desktop/plugins/folder/positioner.h containments/desktop/plugins/folder/previewpluginsmodel.cpp containments/desktop/plugins/folder/previewpluginsmodel.h containments/desktop/plugins/folder/rubberband.h containments/desktop/plugins/folder/shortcut.h containments/desktop/plugins/folder/viewpropertiesmenu.cpp containments/desktop/plugins/folder/viewpropertiesmenu.h To: mlaurent Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart