Review Request 119929: Cleaning up plasma-shell, episode 1

Mon Aug 25 14:53:51 UTC 2014

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119929/
-----------------------------------------------------------

(Updated Aug. 25, 2014, 2:53 p.m.)

Status
------

This change has been marked as submitted.

Review request for Plasma.

Repository: plasma-workspace

Description
-------

While reviewing the plasma-shell binary for use in a device project, I came across a number of issues which are fixed in a longer-than-probably-desirable series of patches in the clean_shellcorona branch of plasma-workspace. There are two types of changes:

* improving multiscreen code; in particular there was a recursive function that would (I assume innadvertently) migrate panels to the last screen when detached. there was also potentially unecessary signal emissions, 
* code cleanups

The code cleanups come in a few flavors:
* getting code in line with the kdelibs coding style this project purportedly follows
* removal of dead code
* checking of parameters where sensible
* making debug code only available in debug builds (c.f. CHECK_SCREEN_INVARIANTS; this would previously result in a bunch of compiler notices when built in release mode)

It is probably easier to review the patches in the branch than the monster review this has become, however many of the patches (though admittedly not all) are built one atop the other making individual review tricky; it was one of those "tug on a loose thread, and the whole sweater came unravelled" type things.

Diffs
-----

  shell/scripting/scriptengine.cpp 94e93fe810edb1743b84cdca168bc7f7e556e982 
  shell/shellcorona.h ebfbdc7ceea9f6cfb387c1931e78faa9e43a894d 
  shell/shellcorona.cpp 29935a51ffc490e1d7e91e2a1109764a4ff2d101 
  shell/widgetexplorer/widgetexplorer.cpp 0ad5cc8519c8f37ce20e49332491ea0469ee7a29 
  shell/containmentconfigview.cpp f489f1771f3de0a0a5fb7e385615e5981b2f240f 
  shell/desktopview.cpp 0f8bde7af9bae1a17ff42d54c9882fbddc695694 

Diff: https://git.reviewboard.kde.org/r/119929/diff/

Testing
-------

Thanks,

Aaron J. Seigo

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140825/bc55d4b8/attachment.html>