Skip to content

Commit 67e5509

Browse files
committed
Replace break statements.
1 parent a15fce1 commit 67e5509

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/ImportOds.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ std::pair<bool, QStringList> ImportOds::retrieveColumnNames(
212212
{
213213
if (isCellStart(reader) &&
214214
(getColumnRepeatCount(reader.attributes()) > 1))
215-
break;
215+
return {true, columnNames};
216216

217217
if ((reader.name().toString() == P_TAG) && reader.isStartElement())
218218
{
@@ -305,7 +305,8 @@ bool ImportOds::moveToSecondRow(const QString& sheetName,
305305
if (isRowStart(reader))
306306
{
307307
if (secondRow)
308-
break;
308+
return true;
309+
309310
secondRow = true;
310311
}
311312
reader.readNext();
@@ -323,7 +324,7 @@ void ImportOds::skipToSheet(QXmlStreamReader& reader,
323324
if (reader.attributes().value(TABLE_NAME_TAG) != sheetName)
324325
reader.skipCurrentElement();
325326
else
326-
break;
327+
return;
327328
}
328329
reader.readNextStartElement();
329330
}

0 commit comments

Comments
 (0)