Review Request 113426: Adjust API in KEmoticons framework: createNew method

Commit Hook null at kde.org
Tue Oct 29 19:01:23 UTC 2013


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


This review has been submitted with commit 89d2b298dfe4b97b2e51d55bb85f96a1e803714e by David Gil to branch frameworks.

- Commit Hook


On Oct. 24, 2013, 9:54 p.m., David Gil Oliva wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113426/
> -----------------------------------------------------------
> 
> (Updated Oct. 24, 2013, 9:54 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Adjust API in KEmoticons framework: createNew method
> 
> -To make KEmoticons API more consistent, deprecate KEmoticonsProvider::createNew()
> and prefer newTheme() instead, as it appears in KEmoticonsTheme. That way,
> we have loadTheme(), saveTheme() and newTheme().
> -Adjust plugins.
> -Before the cleanup, KEmoticonsTheme was calling KEmoticonsProvider::createNew(),
> which was empty. Therefore, I deprecate it and advice subclassing
> KEmoticonsProvider.
> 
> 
> Diffs
> -----
> 
>   KDE5PORTING.html ceff2fa13e4a666939dd0a1bb63e967504c31c07 
>   staging/kemoticons/src/core/kemoticons.cpp 43dac6517b77a0d0040912958fe76687b475d85c 
>   staging/kemoticons/src/core/kemoticonsprovider.h 2ec0de8d1dfb846188bd458b49a4028fee115431 
>   staging/kemoticons/src/core/kemoticonsprovider.cpp 7374966c65922c3e7a5be881c198a8f8f52fee29 
>   staging/kemoticons/src/core/kemoticonstheme.h 25fc29453535d7e73f4e2d0752ce3f989c83fa96 
>   staging/kemoticons/src/core/kemoticonstheme.cpp e54d015e7f0f866d199d8eed7863fafd28576c13 
>   staging/kemoticons/src/providers/adium/adium_emoticons.h 01d89e13834c345765e696d66560071dc10291af 
>   staging/kemoticons/src/providers/adium/adium_emoticons.cpp e6719d112a14478bdfd7f8c47633c18108a5633a 
>   staging/kemoticons/src/providers/kde/kde_emoticons.h 0738b79dcf734b7904e061b5eb41807ccaf443ff 
>   staging/kemoticons/src/providers/kde/kde_emoticons.cpp a99c6d84f5ab7e0e2f41027c37a97f170333dca8 
>   staging/kemoticons/src/providers/pidgin/pidgin_emoticons.h a51b736f7702d7af1f1367dd1f13271647212fee 
>   staging/kemoticons/src/providers/pidgin/pidgin_emoticons.cpp 7596e30e8e5153185a3dd365858567c69477ff4a 
>   staging/kemoticons/src/providers/xmpp/xmpp_emoticons.h 4ba706f519cebedaa6c9c3f2f02331e85745e89a 
>   staging/kemoticons/src/providers/xmpp/xmpp_emoticons.cpp afb07b207407b00bbe0d38e0ca6d9e2bf2ccd809 
> 
> Diff: http://git.reviewboard.kde.org/r/113426/diff/
> 
> 
> Testing
> -------
> 
> Builds. Tests pass.
> 
> 
> Thanks,
> 
> David Gil Oliva
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131029/837c4d2b/attachment.html>


More information about the Kde-frameworks-devel mailing list