Re: [Okular-devel] Review Request 126379: Windows string fixes
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126379/ --- (Updated Dec. 28, 2015, 9:50 p.m.) Status -- This change has been marked as submitted. Review request for kdewin and Okular. Changes --- Submitted with commit d0ecab47368288ec00318dccb6dfcd9fc79a8e8b by Patrick Spendrin to branch frameworks. Repository: okular Description --- This patch changes multiline QStringLiterals to QString::fromLatin1 as QStringLiterals doesn't understand multiline strings on windows. Also, strncasecmp is replaced by the portable qstricmp function provided by QByteArray header. Diffs - generators/dvi/dviRenderer_prescan.cpp 783c7273c6f7d1be03f5e970dc23e0a24a31197f generators/dvi/psgs.cpp b1eedc42864abf5a868a5bc6eebf258b1f7cf168 generators/dvi/special.cpp 6a9fe69b2d44cfed95284b4d37f767c00c73721c generators/epub/converter.cpp 16a6b5beabe1803ea4cdacc4d571edca7ff9ec23 ui/latexrenderer.cpp 8e42a9f88a8aa4ff73b544bf41642d59ca86474a ui/pageview.cpp 13727e4647781cd9222956cae2c9e3fbc1a42324 Diff: https://git.reviewboard.kde.org/r/126379/diff/ Testing --- Windows. Thanks, Patrick Spendrin ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
Re: [Okular-devel] Review Request 126379: Windows string fixes
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126379/#review90227 --- Ship it! I don't really care so i guess ship it, but this will break again once we use the clazy tool that warns us that the QString::fromLatin1 is doing an unneeded QString allocation. So may make sense to either fix QStringLiteral on windows or fix the code in a way it works both on windows and linux without using QString::fromLatin1 - Albert Astals Cid On des. 27, 2015, 12:28 a.m., Patrick Spendrin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126379/ > --- > > (Updated des. 27, 2015, 12:28 a.m.) > > > Review request for kdewin and Okular. > > > Repository: okular > > > Description > --- > > This patch changes multiline QStringLiterals to QString::fromLatin1 > as QStringLiterals doesn't understand multiline strings on windows. > Also, strncasecmp is replaced by the portable qstricmp function provided > by QByteArray header. > > > Diffs > - > > generators/dvi/dviRenderer_prescan.cpp > 783c7273c6f7d1be03f5e970dc23e0a24a31197f > generators/dvi/psgs.cpp b1eedc42864abf5a868a5bc6eebf258b1f7cf168 > generators/dvi/special.cpp 6a9fe69b2d44cfed95284b4d37f767c00c73721c > generators/epub/converter.cpp 16a6b5beabe1803ea4cdacc4d571edca7ff9ec23 > ui/latexrenderer.cpp 8e42a9f88a8aa4ff73b544bf41642d59ca86474a > ui/pageview.cpp 13727e4647781cd9222956cae2c9e3fbc1a42324 > > Diff: https://git.reviewboard.kde.org/r/126379/diff/ > > > Testing > --- > > Windows. > > > Thanks, > > Patrick Spendrin > > ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
Re: [Okular-devel] Review Request 126379: Windows string fixes
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126379/#review90208 --- +1 - Aleix Pol Gonzalez On Dec. 27, 2015, 1:28 a.m., Patrick Spendrin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126379/ > --- > > (Updated Dec. 27, 2015, 1:28 a.m.) > > > Review request for kdewin and Okular. > > > Repository: okular > > > Description > --- > > This patch changes multiline QStringLiterals to QString::fromLatin1 > as QStringLiterals doesn't understand multiline strings on windows. > Also, strncasecmp is replaced by the portable qstricmp function provided > by QByteArray header. > > > Diffs > - > > generators/dvi/dviRenderer_prescan.cpp > 783c7273c6f7d1be03f5e970dc23e0a24a31197f > generators/dvi/psgs.cpp b1eedc42864abf5a868a5bc6eebf258b1f7cf168 > generators/dvi/special.cpp 6a9fe69b2d44cfed95284b4d37f767c00c73721c > generators/epub/converter.cpp 16a6b5beabe1803ea4cdacc4d571edca7ff9ec23 > ui/latexrenderer.cpp 8e42a9f88a8aa4ff73b544bf41642d59ca86474a > ui/pageview.cpp 13727e4647781cd9222956cae2c9e3fbc1a42324 > > Diff: https://git.reviewboard.kde.org/r/126379/diff/ > > > Testing > --- > > Windows. > > > Thanks, > > Patrick Spendrin > > ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
Re: [Okular-devel] Review Request 126379: Windows string fixes
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126379/ --- (Updated Dez. 27, 2015, 12:28 vorm.) Review request for Okular. Repository: okular Description --- This patch changes multiline QStringLiterals to QString::fromLatin1 as QStringLiterals doesn't understand multiline strings on windows. Also, strncasecmp is replaced by the portable qstricmp function provided by QByteArray header. Diffs (updated) - generators/dvi/dviRenderer_prescan.cpp 783c7273c6f7d1be03f5e970dc23e0a24a31197f generators/dvi/psgs.cpp b1eedc42864abf5a868a5bc6eebf258b1f7cf168 generators/dvi/special.cpp 6a9fe69b2d44cfed95284b4d37f767c00c73721c generators/epub/converter.cpp 16a6b5beabe1803ea4cdacc4d571edca7ff9ec23 ui/latexrenderer.cpp 8e42a9f88a8aa4ff73b544bf41642d59ca86474a ui/pageview.cpp 13727e4647781cd9222956cae2c9e3fbc1a42324 Diff: https://git.reviewboard.kde.org/r/126379/diff/ Testing --- Windows. Thanks, Patrick Spendrin ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
[Okular-devel] Review Request 126379: Windows string fixes
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126379/ --- Review request for kdewin and Okular. Repository: okular Description --- This patch changes multiline QStringLiterals to QString::fromLatin1 as QStringLiterals doesn't understand multiline strings on windows. Also, strncasecmp is replaced by the portable qstricmp function provided by QByteArray header. Diffs - generators/dvi/dviRenderer_prescan.cpp 783c7273c6f7d1be03f5e970dc23e0a24a31197f generators/dvi/psgs.cpp b1eedc42864abf5a868a5bc6eebf258b1f7cf168 generators/dvi/special.cpp 6a9fe69b2d44cfed95284b4d37f767c00c73721c generators/epub/converter.cpp 16a6b5beabe1803ea4cdacc4d571edca7ff9ec23 ui/latexrenderer.cpp 8e42a9f88a8aa4ff73b544bf41642d59ca86474a ui/pageview.cpp 13727e4647781cd9222956cae2c9e3fbc1a42324 Diff: https://git.reviewboard.kde.org/r/126379/diff/ Testing --- Windows. Thanks, Patrick Spendrin ___ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel