--===============1245668749== Content-Type: multipart/alternative; boundary="===============6965090167106545767==" --===============6965090167106545767== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100817/ ----------------------------------------------------------- Review request for kwin. Summary ------- setNumberOfDesktops is called from the desktopgrid (or pot. other effects) = and this code belongs there and not into the config loading also the function included some (probably evolutionary) redundant blocks needs backport - i'm however not sure that this really will fix the bug, it= 's just overflow prone and the bug likely defers from an overflow (if the c= laimed Qt bug is none) This addresses bug 200213. http://bugs.kde.org/show_bug.cgi?id=3D200213 Diffs ----- kwin/workspace.cpp 61ce2fe = Diff: http://git.reviewboard.kde.org/r/100817/diff Testing ------- created desktops w/ desktopgrid, moved windows to the end, removed desktops= , w/ & w/o ending the effect in between Thanks, Thomas --===============6965090167106545767== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable
This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/100817/

Review request for kwin.
By Thomas L=C3=BCbking.

Descripti= on 

setNumberOfDesktops is called from the desktopgrid (or pot. =
other effects) and this code belongs there and not into the config loading
also the function included some (probably evolutionary) redundant blocks
needs backport - i'm however not sure that this really will fix the bug=
, it's just overflow prone and the bug likely defers from an overflow (=
if the claimed Qt bug is none)

Testing <= /h1> 
created desktops w/ desktopgrid, moved windows to the end, r=
emoved desktops, w/ & w/o ending the effect in between
Bugs: 200213

Diffs=

  • kwin/workspace.cpp (61ce2fe)

View Diff
--===============6965090167106545767==-- --===============1245668749== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ kwin mailing list kwin@kde.org https://mail.kde.org/mailman/listinfo/kwin --===============1245668749==--