tempo-plotter Changeset - dcd6029a3bce · protofusion repositories

Changeset - dcd6029a3bce

Parent rev.

Child rev.

[Not reviewed]

new-reader

0 3 0

Hasan Yavuz ÖZDERYA - 9 years ago 2016-05-29 13:24:14
hy@ozderya.net

renamed checksumEnabled function

3 files changed with 3 insertions and 3 deletions:

src/framedreader.cpp

src/framedreadersettings.cpp

src/framedreadersettings.h

0 comments (0 inline, 0 general)

src/framedreader.cpp

➞

Show inline comments

@@ @@ -13,49 +13,49 @@ @@
   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
   GNU General Public License for more details.
   You should have received a copy of the GNU General Public License
   along with serialplot.  If not, see <http://www.gnu.org/licenses/>.
 */
 #include <QtDebug>
 #include <QtEndian>
 #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;
     frameSize = _settingsWidget.frameSize();
     syncWord = _settingsWidget.syncWord();
-    checksumEnabled = _settingsWidget.checksumEnabled();
+    checksumEnabled = _settingsWidget.isChecksumEnabled();
     onNumberFormatChanged(_settingsWidget.numberFormat());
     checkSettings();
     connect(&_settingsWidget, &FramedReaderSettings::numberFormatChanged,
             this, &FramedReader::onNumberFormatChanged);
     connect(&_settingsWidget, &FramedReaderSettings::numOfChannelsChanged,
             this, &FramedReader::onNumOfChannelsChanged);
     connect(&_settingsWidget, &FramedReaderSettings::syncWordChanged,
             this, &FramedReader::onSyncWordChanged);
     connect(&_settingsWidget, &FramedReaderSettings::frameSizeChanged,
             this, &FramedReader::onFrameSizeChanged);
     connect(&_settingsWidget, &FramedReaderSettings::checksumChanged,
             [this](bool enabled){checksumEnabled = enabled;});
+}
 void FramedReader::enable(bool enabled)
+{
     if (enabled)
+    {
         // TODO

src/framedreadersettings.cpp

➞

Show inline comments

@@ @@ -110,28 +110,28 @@ QByteArray FramedReaderSettings::syncWor @@
     else
+    {
         return QByteArray::fromHex(text.toLatin1());
+    }
+}
 void FramedReaderSettings::onSyncWordEdited()
+{
     // TODO: emit with a delay so that error message doesn't flash!
     emit syncWordChanged(syncWord());
+}
 unsigned FramedReaderSettings::frameSize()
+{
     if (ui->rbFixedSize->isChecked())
+    {
         return ui->spSize->value();
+    }
     else
+    {
         return 0; // frame byte is enabled
+    }
+}
-bool FramedReaderSettings::checksumEnabled()
+bool FramedReaderSettings::isChecksumEnabled()
+{
     return ui->cbChecksum->isChecked();
+}

src/framedreadersettings.h

➞

Show inline comments

@@ @@ -24,44 +24,44 @@ @@
 #include <QByteArray>
 #include "numberformatbox.h"
 #include "endiannessbox.h"
 namespace Ui {
 class FramedReaderSettings;
+}
 class FramedReaderSettings : public QWidget
+{
     Q_OBJECT
 public:
     explicit FramedReaderSettings(QWidget *parent = 0);
     ~FramedReaderSettings();
     void showMessage(QString message, bool error = false);
     unsigned numOfChannels();
     NumberFormat numberFormat();
     Endianness endianness();
     QByteArray syncWord();
     unsigned frameSize(); /// If frame bye is enabled `0` is returned
-    bool checksumEnabled();
+    bool isChecksumEnabled();
 signals:
     /// If sync word is invalid (empty or 1 nibble missing at the end)
     /// signaled with an empty array
     void syncWordChanged(QByteArray);
     /// `0` indicates frame size byte is enabled
     void frameSizeChanged(unsigned);
     void checksumChanged(bool);
     void numOfChannelsChanged(unsigned);
     void numberFormatChanged(NumberFormat);
 private:
     Ui::FramedReaderSettings *ui;
 private slots:
     void onSyncWordEdited();
 };
 #endif // FRAMEDREADERSETTINGS_H

0 comments (0 inline, 0 general)