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.

Reply via email to