tempo-plotter Changeset - 724d38d30fb1 · protofusion repositories

Changeset - 724d38d30fb1

Parent rev.

Child rev.

[Not reviewed]

new-reader

0 2 0

Hasan Yavuz ÖZDERYA - 10 years ago 2016-05-29 14:16:30
hy@ozderya.net

renamed hasFrameByte to hasSizeByte

2 files changed with 5 insertions and 5 deletions:

src/framedreader.cpp

src/framedreader.h

0 comments (0 inline, 0 general)

src/framedreader.cpp

➞

Show inline comments

@@ @@ -22,25 +22,25 @@ @@
 #include "floatswap.h"
 #include "framedreader.h"
 FramedReader::FramedReader(QIODevice* device, ChannelManager* channelMan, QObject *parent) :
     AbstractReader(device, channelMan, parent)
+{
     paused = false;
     // initial settings
     settingsInvalid = 0;
     _numOfChannels = _settingsWidget.numOfChannels();
-    hasFrameByte = _settingsWidget.frameSize() == 0;
+    hasSizeByte = _settingsWidget.frameSize() == 0;
     frameSize = _settingsWidget.frameSize();
     syncWord = _settingsWidget.syncWord();
     checksumEnabled = _settingsWidget.isChecksumEnabled();
     onNumberFormatChanged(_settingsWidget.numberFormat());
     checkSettings();
     connect(&_settingsWidget, &FramedReaderSettings::numberFormatChanged,
             this, &FramedReader::onNumberFormatChanged);
     connect(&_settingsWidget, &FramedReaderSettings::numOfChannelsChanged,
             this, &FramedReader::onNumOfChannelsChanged);
@@ @@ -144,25 +144,25 @@ void FramedReader::checkSettings() @@
+{
     // sync word is invalid (empty or missing a nibble at the end)
     if (!syncWord.size())
+    {
         settingsInvalid |= SYNCWORD_INVALID;
+    }
     else // sync word is valid
+    {
         settingsInvalid &= ~SYNCWORD_INVALID;
+    }
     // check if fixed frame size is multiple of a sample set size
-    if (!hasFrameByte && frameSize % (_numOfChannels * sampleSize) != 0)
+    if (!hasSizeByte && frameSize % (_numOfChannels * sampleSize) != 0)
+    {
         settingsInvalid |= FRAMESIZE_INVALID;
+    }
     else
+    {
         settingsInvalid &= ~FRAMESIZE_INVALID;
+    }
     // show an error message
     if (settingsInvalid & SYNCWORD_INVALID)
+    {
         _settingsWidget.showMessage("Sync word is invalid!", true);
@@ @@ -188,21 +188,21 @@ void FramedReader::onNumOfChannelsChange @@
+}
 void FramedReader::onSyncWordChanged(QByteArray word)
+{
     syncWord = word;
     checkSettings();
+}
 void FramedReader::onFrameSizeChanged(unsigned value)
+{
     if (value == 0)
+    {
-        hasFrameByte = true;
+        hasSizeByte = true;
+    }
     else
+    {
-        hasFrameByte = false;
+        hasSizeByte = false;
         frameSize = value;
+    }
     checkSettings();
+}

src/framedreader.h

➞

Show inline comments

@@ @@ -45,25 +45,25 @@ private: @@
+    {
         SYNCWORD_INVALID = 1,
         FRAMESIZE_INVALID = 2
     };
     FramedReaderSettings _settingsWidget;
     unsigned _numOfChannels;
     unsigned sampleSize;
     bool paused;
     unsigned settingsInvalid;   /// settings are all valid if this is 0, if not no reading is done
     QByteArray syncWord;
     bool checksumEnabled;
-    bool hasFrameByte;
+    bool hasSizeByte;
     unsigned frameSize;
     // TODO: create a base class that contains common functionality between binary readers
     /// points to the readSampleAs function for currently selected number format
     double (FramedReader::*readSample)();
     template<typename T> double readSampleAs();
     // `data` contains i th channels data
     void addChannelData(unsigned int channel, double* data, unsigned size);
     /// Checks the validity of syncWord and frameSize then shows an
     /// error message. Also updates `settingsInvalid`. If settings are

0 comments (0 inline, 0 general)