https://bugs.kde.org/show_bug.cgi?id=369665
Marco Martin <notm...@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Latest Commit| |http://commits.kde.org/plas | |ma-workspace/ca1bf0e3e4c018 | |127d2aac0a78a2e44385b76174 Resolution|--- |FIXED --- Comment #19 from Marco Martin <notm...@gmail.com> --- Git commit ca1bf0e3e4c018127d2aac0a78a2e44385b76174 by Marco Martin. Committed on 10/10/2016 at 14:53. Pushed by mart into branch 'Plasma/5.8'. fix screenForContainment when screen ids > count Summary: the check that was there, screen id not being more than screen count is not valid anymore with the screenpool approach, a screen id can be bigger than the screen count in cases such as middle screen of a 3 screen system disabled, or driver change (in which unfortunately, connector names can change) without this, a new containment was created for each startup. Test Plan: created a puroposefully corrupted plasmashellrc file, where one known screen has a big number as id. without patch a new containment is added in appletsrc each time with the patch it behaves fine. Reviewers: #plasma, davidedmundson Reviewed By: #plasma, davidedmundson Subscribers: davidedmundson, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D3006 M +4 -3 shell/shellcorona.cpp http://commits.kde.org/plasma-workspace/ca1bf0e3e4c018127d2aac0a78a2e44385b76174 -- You are receiving this mail because: You are watching all bug changes.