Review Request 120571: Port KAppTemplate away from KDELibs4Support

[Milian Wolff]

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

Ship it!


two small nitpicks, then go ahead and push it - thanks!


generatepage.cpp
<https://git.reviewboard.kde.org/r/120571/#comment47659>

    style: join line and add spaces (also below). I know it was wrong before, but please change it while at it
    
        if (...) {
            ...
        } else if (...) {
            ...
        }



generatepage.cpp
<https://git.reviewboard.kde.org/r/120571/#comment47660>

    '/'


- Milian Wolff


On Oct. 13, 2014, 6:24 p.m., Simon Wächter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120571/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2014, 6:24 p.m.)
> 
> 
> Review request for KDevelop and Jonathan Riddell.
> 
> 
> Repository: kapptemplate
> 
> 
> Description
> -------
> 
> This patch drops the KDE4 library dependency:
> 
> - Removed the KDE4 library and 3 other CMake modules that where triggered by the support library
> - Fixed the includes and removed the old ones
> - Switched from kDebug to qCDebug
> - Rewritten copy and macro expanding
> - Removed a (unused) signal that tries to override the same signal in the base class (Results in warning)
> - Removed whitespaces
> 
> Please check:
> - Take a look at generatepage.cpp --> unpackArchive and generatepage.cpp --> finishFile. Is it a good idea to extract all macros in a file - even when it is a "binary blob" like an ODF or binary etc. ? Does someone know a good way to solve this ? In the end I always get stuck in a static file ending or mime type check (But there are some many minetypes ...)
> 
> Future:
> - Move the functionality inside a library, so that KAppTemplate and later KDevelop can use it to find file and project templates
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 4ea08d4 
>   apptemplatesmodel.cpp 2ef00d7 
>   choicepage.cpp 7ae478f 
>   generatepage.h 22085d1 
>   generatepage.cpp 66b98ab 
>   kapptemplate.h 77f43c0 
>   kapptemplate.cpp b1abcab 
>   logging.h PRE-CREATION 
>   main.cpp 038238e 
> 
> Diff: https://git.reviewboard.kde.org/r/120571/diff/
> 
> 
> Testing
> -------
> 
> Testing was done under a project neon 5 system
> 
> 
> Thanks,
> 
> Simon Wächter
> 
>

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