sitter updated this revision to Diff 75170. sitter added a comment.
move from qmap to straight up qlist as values() within the map wouldn't necessarily have the expected order. instead find_by udi on add/remove and rely on index within the list the rest of the time additional qa: have 2 mounts, mount/umount randomly REPOSITORY R102 KInfoCenter CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27061?vs=74992&id=75170 BRANCH smb REVISION DETAIL https://phabricator.kde.org/D27061 AFFECTED FILES LICENSES/GPL-2.0-only.txt LICENSES/GPL-2.0-or-later.txt LICENSES/GPL-3.0-only.txt LICENSES/LicenseRef-KDE-Accepted-GPL.txt LICENSES/MIT.txt Modules/samba/CMakeLists.txt Modules/samba/autotests/CMakeLists.txt Modules/samba/autotests/ksambasharemodeltest.cpp Modules/samba/autotests/smbmountmodeltest.cpp Modules/samba/kcmsambaimports.cpp Modules/samba/kcmsambaimports.h Modules/samba/kcmsambalog.cpp Modules/samba/kcmsambalog.h Modules/samba/kcmsambastatistics.cpp Modules/samba/kcmsambastatistics.h Modules/samba/ksambasharemodel.cpp Modules/samba/ksambasharemodel.h Modules/samba/ksmbstatus.cpp Modules/samba/ksmbstatus.h Modules/samba/main.cpp Modules/samba/smbmountmodel.cpp Modules/samba/smbmountmodel.h To: sitter, #localization, #plasma, #vdg Cc: jriddell, davidedmundson, ltoscano, yurchor, ngraham, alexde, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart