Review Request:

[Chani]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/637/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

this is just the code to read shortcuts from the rc file. it works, but I'm not 100% happy with it - that function added to Applet feels awkward and its name sucks, and I can't help wondering if maybe I should put it somewhere else, or duplicate it in containment so that it can be private... any thoughts?


Diffs
-----

  /trunk/KDE/kdelibs/plasma/applet.h 960161 
  /trunk/KDE/kdelibs/plasma/applet.cpp 960161 
  /trunk/KDE/kdelibs/plasma/containment.cpp 960161 
  /trunk/KDE/kdelibs/plasma/corona.cpp 960161 

Diff: http://reviewboard.kde.org/r/637/diff


Testing
-------


Thanks,

Chani