Review Request 126531: Fix some Clazy warnings in kemoticons.

Andrey Cygankov craftplace.ms at gmail.com
Sun Dec 27 17:50:14 UTC 2015


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

(Updated Dec. 27, 2015, 6:50 p.m.)


Review request for KDE Frameworks and Aleix Pol Gonzalez.


Repository: kemoticons


Description
-------

Fix some Clazy warnings:
- unneeded heap allocation with QString
- multi-arg non-usage


Diffs
-----

  autotests/kemoticontest.cpp df7fb71 
  autotests/ktexttohtmlplugintest.cpp 0daa222 
  src/core/kemoticons.cpp 886cc3f 
  src/providers/adium/adium_emoticons.cpp 31535ab 
  src/providers/kde/kde_emoticons.cpp 2723e00 
  src/providers/pidgin/pidgin_emoticons.cpp 92398c4 
  src/providers/xmpp/xmpp_emoticons.cpp af71ef9 
  tests/main.cpp d1eab5b 

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


Testing
-------

Compiling without errors.


Thanks,

Andrey Cygankov

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151227/0d9595ff/attachment.html>


More information about the Kde-frameworks-devel mailing list