Review Request 112457: Start cleanup of KDocTools

Aleix Pol Gonzalez aleixpol at kde.org
Tue Sep 3 12:48:57 UTC 2013



> On Sept. 3, 2013, 9:30 a.m., David Faure wrote:
> > staging/kdoctools/src/meinproc.cpp, line 146
> > <http://git.reviewboard.kde.org/r/112457/diff/1/?file=186392#file186392line146>
> >
> >     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.

Ok, I have it fixed locally, I'll push it once we have a way to find DocBook from ECM.


- Aleix


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


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/fa38be89/attachment.html>


More information about the Kde-frameworks-devel mailing list