> 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 > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
