Review Request 113086: KDeclarative cleanup

[Aurélien Gâteau]

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


Nothing serious, just some nitpicking.


experimental/kdeclarative/bindings/backportglobal.h
<http://git.reviewboard.kde.org/r/113086/#comment30236>

    missing spaces in `typeName.size()-1` and trailing space after .const.
    
    Would be more readable to move typeName.left(...) to its own var.



experimental/kdeclarative/qmlobject.cpp
<http://git.reviewboard.kde.org/r/113086/#comment30235>

    This is not exactly the same: if I am not mistaken, qWarning() will add extra spaces and maybe quotes around the '\n'.


- Aurélien Gâteau


On Oct. 3, 2013, 10:02 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113086/
> -----------------------------------------------------------
> 
> (Updated Oct. 3, 2013, 10:02 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> Reorganizes the file structure as the KF5 module suggest.
> Adopts newer syntaxes for defining dependencies.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 3204c11 
>   experimental/CMakeLists.txt e73a8c4 
>   experimental/COPYING 5185fd3 
>   experimental/COPYING.LIB 2d2d780 
>   experimental/CTestConfig.cmake c383ee1 
>   experimental/KDELibsExperimentalNightly.cmake fd53e0c 
>   experimental/kdeclarative/CMakeLists.txt 72ea7a2 
>   experimental/kdeclarative/bindings/backportglobal.h fca8ade 
>   experimental/kdeclarative/bindings/i18n.cpp 75a67aa 
>   experimental/kdeclarative/bindings/i18n_p.h  
>   experimental/kdeclarative/bindings/icon.cpp dbdc8fe 
>   experimental/kdeclarative/bindings/qscriptnonguibookkeeping.cpp 919e889 
>   experimental/kdeclarative/bindings/url.cpp 3031a6f 
>   experimental/kdeclarative/config-kdeclarative.h.cmake 611c678 
>   experimental/kdeclarative/configpropertymap.h c9ef3ae 
>   experimental/kdeclarative/configpropertymap.cpp 373a655 
>   experimental/kdeclarative/kdeclarative.h  
>   experimental/kdeclarative/kdeclarative.cpp 63ae180 
>   experimental/kdeclarative/kdeclarativeConfig.cmake.in d68008a 
>   experimental/kdeclarative/kdeclarativeConfigVersion.cmake.in eae2225 
>   experimental/kdeclarative/private/kdeclarative_p.h  
>   experimental/kdeclarative/private/kiconprovider.cpp 8506b8b 
>   experimental/kdeclarative/private/kiconprovider_p.h  
>   experimental/kdeclarative/private/rootcontext.cpp  
>   experimental/kdeclarative/private/rootcontext_p.h  
>   experimental/kdeclarative/qmlobject.h  
>   experimental/kdeclarative/qmlobject.cpp ee81b41 
>   experimental/kdeclarative/qmlobject_p.h  
>   experimental/kdeclarative/test/test.cpp 3986b77 
>   experimental/kdeclarative/test/test.qml 060c265 
>   experimental/kdeclarative/test/testobject_p.h  
>   staging/CMakeLists.txt 5c5c80c 
>   staging/kdeclarative/CMakeLists.txt PRE-CREATION 
>   staging/kdeclarative/KDeclarativeConfig.cmake.in PRE-CREATION 
>   staging/kdeclarative/src/CMakeLists.txt PRE-CREATION 
>   staging/kdeclarative/tests/CMakeLists.txt PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/113086/diff/
> 
> 
> Testing
> -------
> 
> Builds.
> There's a test that tests things that are not part either of this module or its dependencies:
> 
> import Qt 4.7
> import org.kde.plasma.core 0.1 as PlasmaCore
> import org.kde.plasma.graphicswidgets 0.1 as PlasmaWidgets
> 
> I'm unsure what to do with those. (can do in a further commit, we're not moving the module to its tier yet)
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131004/1ded3dc6/attachment.html>