Review Request 112457: Start cleanup of KDocTools
David Faure
faure at kde.org
Tue Sep 3 09:30:20 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112457/#review39242
-----------------------------------------------------------
Ship it!
just one thing
staging/kdoctools/config-kdoctools.h.cmake
<http://git.reviewboard.kde.org/r/112457/#comment28940>
It's a bit strange to do that here.
See below.
staging/kdoctools/src/meinproc.cpp
<http://git.reviewboard.kde.org/r/112457/#comment28939>
I'd write exe = QStringLiteral(XMLLINT)
instead of putting the QStringLiteral in the config-foo.h file (which doesn't know how it's going to be used).
The preprocessor will replace the macro with the literal so QStringLiteral is correct here.
- David Faure
On Sept. 2, 2013, 4:42 p.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112457/
> -----------------------------------------------------------
>
> (Updated Sept. 2, 2013, 4:42 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Description
> -------
>
> Start using DocBook*cmake modules from ECM (pending to be solved).
> Properly construct strings for KDocTools, making sure QStringLiteral & friends are used.
>
>
> Diffs
> -----
>
> staging/kdoctools/config-kdoctools.h.cmake f2fe22c
> staging/kdoctools/src/CMakeLists.txt de82d4d
> staging/kdoctools/src/docbookl10nhelper.cpp abd980c
> staging/kdoctools/src/meinproc.cpp 1c72881
> staging/kdoctools/src/meinproc_common.cpp 18f7346
> staging/kdoctools/src/meinproc_simple.cpp ad34fb9
> staging/kdoctools/src/xslt.cpp 2256bf6
>
> Diff: http://git.reviewboard.kde.org/r/112457/diff/
>
>
> Testing
> -------
>
> Builds, KDocTools has no tests.
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130903/fd1e0503/attachment.html>
More information about the Kde-frameworks-devel
mailing list