Review Request 126400: [GCI] Fix some Clazy warnings
    Imran Tatriev 
    itatriev at gmail.com
       
    Thu Dec 17 14:34:42 UTC 2015
    
    
  
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126400/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kconfig
Description
-------
I've fixed some warnings, here they are:
- Unneeded heap allocation with QString
- Detaching temporary
- Multi-arg non-usage
- isEmpty() non-usage
- midRef() non-usage
- "QVariant::toFoo() instead of QVariant::value<Foo>()" non-usage
Diffs
-----
  autotests/kconfig_compiler/kconfigcompiler_test.cpp 231e3c4 
  autotests/kconfig_compiler/kconfigcompiler_test_signals.cpp 9a26ca1 
  src/core/kconfiggroup.cpp e09d602 
  src/core/kemailsettings.cpp 59d5f6e 
  src/gui/kconfiggroupgui.cpp b8d3089 
  src/kconf_update/kconf_update.cpp 9567c2d 
  src/kconf_update/kconfigutils.cpp 12b0749 
  src/kconfig_compiler/kconfig_compiler.cpp 91f6005 
  src/kreadconfig/kwriteconfig.cpp a852c28 
Diff: https://git.reviewboard.kde.org/r/126400/diff/
Testing
-------
Thanks,
Imran Tatriev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151217/b5762e96/attachment.html>
    
    
More information about the Kde-frameworks-devel
mailing list