[Panel-devel] JJ: taskmanager clean up
Aaron J. Seigo
aseigo at kde.org
Wed Dec 19 22:47:22 CET 2007
hey party plasma people =)
so ... i had some ganders at workspace/libs/taskmanager today and now i know
what that sound under my desk is: the baby jesus wheeping from the horrors
found within.
there is a bunch of housecleaning that needs to get done in there that would
be just *great* for someone who's looking to cut their teeth on things. in
particular:
* the classes should go into a namespace. exporting a class called Task into
the global namespace is just irresponsible =)
* there are three header files. two of them are context menus. yes, that means
that there are a bunch of classes in the final one: taskmanager.h.
taskmanager.h and taskmanager.cpp should be split up to contain one class per
file.
* speaking of classes hidden in taskmanager.h, who knew there was a drag
object in there, huh? =) heck it even has comments from libkicker days. i
totally forgot about that one even. it should almost certainly be merged into
Task as per the comment.
* CamelCaseIncludes need to be created and installed. see
libs/plasma/includes/ for how this is done
* TaskManager::self() should be updated to use the same method as seen in, for
instance, Plasma::Theme::self()
* TaskManager's remaining static member should be moved into the private class
for cleanliness
* some general code and API review might be in order. things can always be
improved right? =)
anyone feeling like they'd like to tackle one or more of the above? get your
patches ready! =)
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20071219/ec0dfba1/attachment-0001.pgp
More information about the Panel-devel
mailing list