Review Request 117656: Improved KDevelop teplates

Sven Brauch svenbrauch at googlemail.com
Sun Apr 20 19:45:34 UTC 2014


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

Ship it!


All proposed changes clearly look like improvements to me, so I'd say go for it!

- Sven Brauch


On April 20, 2014, 7:34 p.m., Simon Wächter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117656/
> -----------------------------------------------------------
> 
> (Updated April 20, 2014, 7:34 p.m.)
> 
> 
> Review request for KDevelop and Sven Brauch.
> 
> 
> Repository: kdevelop
> 
> 
> Description
> -------
> 
> This patch improves the template provided by KDevelop:
> 
> - Variable naming
> - Indentation
> - Included files
> - CMake files (Version, install commands and readability)
> 
> If there are other things to improve in the templates, please let it know me.
> 
> 
> Diffs
> -----
> 
>   app_templates/cpp/CMake/cmake_plaincpp/CMakeLists.txt f649c5f 
>   app_templates/cpp/CMake/cmake_qt4core/%{APPNAME}.h e7c7b49 
>   app_templates/cpp/CMake/cmake_qt4core/CMakeLists.txt 4e91282 
>   app_templates/cpp/CMake/cmake_qt4core/main.cpp 426830e 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/%{APPNAME}.h 2d49681 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/CMakeLists.txt 5e83972 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/main.cpp 426830e 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/tests/%{APPNAME}Test.h 02759ec 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/tests/%{APPNAME}Test.cpp 632a3f8 
>   app_templates/cpp/CMake/cmake_qt4core_qtest/tests/CMakeLists.txt a07e256 
>   app_templates/cpp/CMake/cmake_qt4gui/%{APPNAME}.h 3c50df5 
>   app_templates/cpp/CMake/cmake_qt4gui/%{APPNAME}.cpp a1ded84 
>   app_templates/cpp/CMake/cmake_qt4gui/CMakeLists.txt c38bdde 
>   app_templates/cpp/CMake/cmake_qt4gui/main.cpp 67c6ca7 
> 
> Diff: https://git.reviewboard.kde.org/r/117656/diff/
> 
> 
> Testing
> -------
> 
> Templates were succesfully tested with gcc 4.8.2 and clang 3.3 under Debian
> 
> 
> Thanks,
> 
> Simon Wächter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140420/d0a19e5b/attachment.html>


More information about the KDevelop-devel mailing list