Skip to content

Commit fae182e

Browse files
committed
remove dependency on git line end conversions
consistently use QIODevice::Text
1 parent 91c0956 commit fae182e

File tree

4 files changed

+17
-15
lines changed

4 files changed

+17
-15
lines changed

tests/auto/linguist/lconvert/tst_lconvert.cpp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ void tst_lconvert::doCompare(QIODevice *actualDev, const QString &expectedFn)
105105
QList<QByteArray> actual = actualDev->readAll().split('\n');
106106

107107
QFile file(expectedFn);
108-
QVERIFY(file.open(QIODevice::ReadOnly));
108+
QVERIFY(file.open(QIODevice::ReadOnly | QIODevice::Text));
109109
QList<QByteArray> expected = file.readAll().split('\n');
110110

111111
int i = 0, ei = expected.size(), gi = actual.size();
@@ -179,7 +179,7 @@ void tst_lconvert::convertChain(const QString &_inFileName, const QString &_outF
179179
if (!argList.isEmpty())
180180
args += argList[i];
181181
args << "-if" << stations[i] << "-i" << "-" << "-of" << stations[i + 1];
182-
cvts.at(i)->start(binDir + "/lconvert", args);
182+
cvts.at(i)->start(binDir + "/lconvert", args, QIODevice::ReadWrite | QIODevice::Text);
183183
}
184184
int st = 0;
185185
foreach (QProcess *cvt, cvts)
@@ -243,7 +243,9 @@ void tst_lconvert::converts()
243243
QString outFileNameFq = dataDir + outFileName;
244244

245245
QProcess cvt;
246-
cvt.start(binDir + "/lconvert", QStringList() << "-i" << (dataDir + inFileName) << "-of" << format);
246+
cvt.start(binDir + "/lconvert",
247+
QStringList() << "-i" << (dataDir + inFileName) << "-of" << format,
248+
QIODevice::ReadWrite | QIODevice::Text);
247249
doWait(&cvt, 0);
248250
if (QTest::currentTestFailed())
249251
return;

tests/auto/linguist/lrelease/tst_lrelease.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,8 @@ private slots:
7070
void tst_lrelease::doCompare(const QStringList &actual, const QString &expectedFn)
7171
{
7272
QFile file(expectedFn);
73-
QVERIFY(file.open(QIODevice::ReadOnly));
74-
QStringList expected = QString(file.readAll()).trimmed().remove('\r').split('\n');
73+
QVERIFY(file.open(QIODevice::ReadOnly | QIODevice::Text));
74+
QStringList expected = QString(file.readAll()).trimmed().split('\n');
7575

7676
int i = 0, ei = expected.size(), gi = actual.size();
7777
for (; ; i++) {
@@ -200,10 +200,10 @@ void tst_lrelease::compressed()
200200
void tst_lrelease::dupes()
201201
{
202202
QProcess proc;
203-
proc.start(binDir + "/lrelease testdata/dupes.ts");
203+
proc.start(binDir + "/lrelease testdata/dupes.ts", QIODevice::ReadWrite | QIODevice::Text);
204204
QVERIFY(proc.waitForFinished());
205205
QVERIFY(proc.exitStatus() == QProcess::NormalExit);
206-
doCompare(QString(proc.readAllStandardError()).trimmed().remove('\r').split('\n'), "testdata/dupes.errors");
206+
doCompare(QString(proc.readAllStandardError()).trimmed().split('\n'), "testdata/dupes.errors");
207207
}
208208

209209
QTEST_MAIN(tst_lrelease)

tests/auto/linguist/lupdate/testlupdate.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,9 @@ bool TestLUpdate::runChild( bool showOutput, const QString &program, const QStri
9898
connect(childProc, SIGNAL(finished(int)), this, SLOT(childReady(int)));
9999
childProc->setProcessChannelMode(QProcess::MergedChannels);
100100
if (argList.isEmpty()) {
101-
childProc->start( program );
101+
childProc->start( program, QIODevice::ReadWrite | QIODevice::Text );
102102
} else {
103-
childProc->start( program, argList );
103+
childProc->start( program, argList, QIODevice::ReadWrite | QIODevice::Text );
104104
}
105105
bool ok;
106106

tests/auto/linguist/lupdate/tst_lupdate.cpp

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ private slots:
8080
void tst_lupdate::doCompare(const QStringList &actual, const QString &expectedFn, bool err)
8181
{
8282
QFile file(expectedFn);
83-
QVERIFY(file.open(QIODevice::ReadOnly));
84-
QStringList expected = QString(file.readAll()).trimmed().remove('\r').split('\n');
83+
QVERIFY(file.open(QIODevice::ReadOnly | QIODevice::Text));
84+
QStringList expected = QString(file.readAll()).trimmed().split('\n');
8585

8686
int i = 0, ei = expected.size(), gi = actual.size();
8787
for (; ; i++) {
@@ -130,8 +130,8 @@ void tst_lupdate::doCompare(const QStringList &actual, const QString &expectedFn
130130
void tst_lupdate::doCompare(const QString &actualFn, const QString &expectedFn, bool err)
131131
{
132132
QFile afile(actualFn);
133-
QVERIFY(afile.open(QIODevice::ReadOnly));
134-
QStringList actual = QString(afile.readAll()).trimmed().remove('\r').split('\n');
133+
QVERIFY(afile.open(QIODevice::ReadOnly | QIODevice::Text));
134+
QStringList actual = QString(afile.readAll()).trimmed().split('\n');
135135

136136
doCompare(actual, expectedFn, err);
137137
}
@@ -169,7 +169,7 @@ void tst_lupdate::good()
169169
QString lupdatecmd;
170170
QFile file(dir + "/lupdatecmd");
171171
if (file.exists()) {
172-
QVERIFY(file.open(QIODevice::ReadOnly));
172+
QVERIFY(file.open(QIODevice::ReadOnly | QIODevice::Text));
173173
while (!file.atEnd()) {
174174
QByteArray cmdstring = file.readLine().simplified();
175175
if (cmdstring.startsWith('#'))
@@ -215,7 +215,7 @@ void tst_lupdate::output_ts()
215215
QString lupdatecmd;
216216
QFile file(dir + "/lupdatecmd");
217217
if (file.exists()) {
218-
QVERIFY(file.open(QIODevice::ReadOnly));
218+
QVERIFY(file.open(QIODevice::ReadOnly | QIODevice::Text));
219219
while (!file.atEnd()) {
220220
QByteArray cmdstring = file.readLine().simplified();
221221
if (cmdstring.startsWith('#'))

0 commit comments

Comments
 (0)