KEmoticons splitting task
David Gil Oliva
davidgiloliva at gmail.com
Mon Aug 26 21:36:13 UTC 2013
Hello,
I took the KEmoticons splitting task. I'm studying the code and I have
found many issues:
--KEmoticonsProvider should be abstract, since its virtual methods either
don't do anything or their code is not safe.
--Many KEmoticons tests don't pass (I haven't actually changed anything
yet). And kdelibs-frameworks/staging/kemoticons/tests/main.cpp doesn't
parse correctly the emoticons. It says 'kemoticonstest(2650)/default
KEmoticonsPrivate::loadProvider: Invalid plugin factory for
"emoticonstheme_kde"'. I suspect that the uses of KPluginLoader have to be
substituted by QPluginLoader in order to work correctly. Could I be right?
--The API method names could be improved to be clearer. I don't know
whether to deprecate the old ones or simply substitute them. What's better?
--KEmoticonsProvider includes <kio/copyjob.h> and <kio/jobuidelegate.h>. Is
this OK?
First I'll try to fix the
kdelibs-frameworks/staging/kemoticons/tests/main.cpp test to be able to
know where the problems are when parsing. Then I'll modify the framework to
be as well written as I can and to have all tests passed. Finally I'll
split it into tier3 folder. I'll send little patches to RB.
Cheers,
David Gil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130826/c5eaaad7/attachment.html>
More information about the Kde-frameworks-devel
mailing list