tempo-plotter Changeset - 06e596e43cde · protofusion repositories

Changeset - 06e596e43cde

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Hasan Yavuz ÖZDERYA - 8 years ago 2018-03-29 17:23:28
hy@ozderya.net

fix issue #15 snapshots not loading completely in some cases

I think this is a buffering issue caused by use of QFile.canReadLine function.
Note that loaded portion of the file was approximately 16k.

1 file changed with 5 insertions and 2 deletions:

src/snapshotmanager.cpp

0 comments (0 inline, 0 general)

src/snapshotmanager.cpp

➞

Show inline comments

@@ @@ -22,6 +22,7 @@ @@
 #include <QKeySequence>
 #include <QFileDialog>
 #include <QFile>
 #include <QTextStream>
 #include <QVector>
 #include <QPointF>
 #include <QIcon>
@@ @@ -159,17 +160,19 @@ void SnapshotManager::loadSnapshotFromFi @@
     // read data
     QVector<QVector<QPointF>> data(numOfChannels);
     QTextStream ts(&file);
     QString line;
     unsigned lineNum = 1;
-    while (file.canReadLine())
+    while (ts.readLineInto(&line))
+    {
         // parse line
         auto line = QString(file.readLine());
         auto split = line.split(',');
         if (split.size() != (int) numOfChannels)
+        {
             qCritical() << "Parsing error at line " << lineNum
                         << ": number of columns is not consistent.";
             qCritical() << "Line " << lineNum << ": " << line;
             return;
+        }

0 comments (0 inline, 0 general)