Review Request 118666: Simple Cleanups

[Aleix Pol Gonzalez]

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



src/core/kconfiggroup.cpp
<https://git.reviewboard.kde.org/r/118666/#comment41656>

    shouldn't this be translated?



src/kconf_update/kconf_update.cpp
<https://git.reviewboard.kde.org/r/118666/#comment41657>

    Why do you change Q_FOREACH to foreach?



src/kconf_update/kconf_update.cpp
<https://git.reviewboard.kde.org/r/118666/#comment41658>

    Doesn't that make the KConfig go out of scope too early?
    



src/kconf_update/kconfigutils.cpp
<https://git.reviewboard.kde.org/r/118666/#comment41660>

    This modifies the actual semantic of the function, although it might still be correct.



src/kconf_update/kconfigutils.cpp
<https://git.reviewboard.kde.org/r/118666/#comment41659>

    "" -> QString()


- Aleix Pol Gonzalez


On June 11, 2014, 1:59 p.m., Thomas Braxton wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118666/
> -----------------------------------------------------------
> 
> (Updated June 11, 2014, 1:59 p.m.)
> 
> 
> Review request for KDE Frameworks and Matthew Dawson.
> 
> 
> Repository: kconfig
> 
> 
> Description
> -------
> 
> Simple Cleanups
> - use Q_NULLPTR instead of 0 or NULL
> - simplify some foreach loops
> - use QStringLiteral in a few places
> - added a few consts
> 
> 
> Diffs
> -----
> 
>   autotests/kconfigloadertest.cpp 451d636e4e584a97d897634b1db1459b47e12355 
>   src/core/kconfig.h d27eebe7c41cb433b1808882c53cbf7b4c870950 
>   src/core/kconfig.cpp ea9746c001e235529a1cdd5865b9e1b5c129b56a 
>   src/core/kconfigbackend.cpp 90655facb49ebb1702fa1635ae5db3fc11fc31a3 
>   src/core/kconfigdata.h e57becb24936865d009a700892893049b3e079f1 
>   src/core/kconfiggroup.h 3c4bce8433e3c5d4cb2d9fdd111a43f04cf3c295 
>   src/core/kconfiggroup.cpp 6f609baefec5beaf38fdfedd6d192b395e3f8acb 
>   src/core/kconfigini.cpp df834f57fc44bbf9f4f28f1bc4eb5717e2ff1cee 
>   src/core/kcoreconfigskeleton.h bb9c1cf936b87e2456726a2bb3428be42558b39f 
>   src/gui/kconfiggroupgui.cpp a14c204c143b38189f042c37ccbe1bf6355a2a18 
>   src/gui/kconfiggui.cpp 1eee8ce50fd660f11cad53c2a493072317e1f331 
>   src/gui/kconfigloader.h 36eb182fbf1c241a043566a13d7c6c123a6e455f 
>   src/gui/kconfigloader.cpp 1dd9e7fc44a367165fedc2e7760c8b524ecd210e 
>   src/gui/kconfigskeleton.h f87eef6e5268a3b4e93fd733f70a2bdd4c4be753 
>   src/kconf_update/kconf_update.cpp 829d35e61da3d303539044d0dc652cc99dec35a2 
>   src/kconf_update/kconfigutils.cpp f2663e1316ec2cdbef303385787d14406e02f4a1 
> 
> Diff: https://git.reviewboard.kde.org/r/118666/diff/
> 
> 
> Testing
> -------
> 
> All tests still pass
> 
> 
> Thanks,
> 
> Thomas Braxton
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140611/b8982f88/attachment-0001.html>