Fix a few minor mistakes

5 files changed, 13 insertions, 14 deletions

diff --git a/src/DabModulator.cpp b/src/DabModulator.cpp
index 0818f4f..8ba8af6 100644
--- a/src/DabModulator.cpp
+++ b/src/DabModulator.cpp
@@ -240,7 +240,7 @@ int DabModulator::process(Buffer* dataOut)
         ////////////////////////////////////////////////////////////////
         // Data initialisation
         ////////////////////////////////////////////////////////////////
-        myFicSizeIn = fic->getFramesize();
+        size_t ficSizeIn = fic->getFramesize();
 
         ////////////////////////////////////////////////////////////////
         // Modules configuration
@@ -252,10 +252,10 @@ int DabModulator::process(Buffer* dataOut)
         PDEBUG(" Framesize: %zu\n", fic->getFramesize());
 
         // Configuring prbs generator
-        auto ficPrbs = make_shared<PrbsGenerator>(myFicSizeIn, 0x110);
+        auto ficPrbs = make_shared<PrbsGenerator>(ficSizeIn, 0x110);
 
         // Configuring convolutionnal encoder
-        auto ficConv = make_shared<ConvEncoder>(myFicSizeIn);
+        auto ficConv = make_shared<ConvEncoder>(ficSizeIn);
 
         // Configuring puncturing encoder
         auto ficPunc = make_shared<PuncturingEncoder>();
diff --git a/src/DabModulator.h b/src/DabModulator.h
index 6878853..2d7bc35 100644
--- a/src/DabModulator.h
+++ b/src/DabModulator.h
@@ -72,6 +72,7 @@ protected:
     size_t myNullSize;
     size_t mySymSize;
     size_t myFicSizeOut;
-    size_t myFicSizeIn;
+
+    std::shared_ptr<OutputMemory> myOutput;
 };
 
diff --git a/src/FrequencyInterleaver.h b/src/FrequencyInterleaver.h
index 7c0fb22..43ca21a 100644
--- a/src/FrequencyInterleaver.h
+++ b/src/FrequencyInterleaver.h
@@ -36,16 +36,14 @@ class FrequencyInterleaver : public ModCodec
 public:
     FrequencyInterleaver(size_t mode);
     virtual ~FrequencyInterleaver();
-    FrequencyInterleaver(const FrequencyInterleaver&);
-    FrequencyInterleaver& operator=(const FrequencyInterleaver&);
+    FrequencyInterleaver(const FrequencyInterleaver&) = delete;
+    FrequencyInterleaver& operator=(const FrequencyInterleaver&) = delete;
 
-
-    int process(Buffer* const dataIn, Buffer* dataOut);
-    const char* name() { return "FrequencyInterleaver"; }
+    int process(Buffer* const dataIn, Buffer* dataOut) override;
+    const char* name() override { return "FrequencyInterleaver"; }
 
 protected:
     size_t d_carriers;
-    size_t d_num;
     size_t* d_indexes;
 };
 
diff --git a/src/RemoteControl.h b/src/RemoteControl.h
index 013738b..11cd52c 100644
--- a/src/RemoteControl.h
+++ b/src/RemoteControl.h
@@ -94,7 +94,7 @@ class BaseRemoteController {
 class RemoteControllable {
     public:
         RemoteControllable(const std::string& name) :
-            m_name(name) {}
+            m_rc_name(name) {}
 
         RemoteControllable(const RemoteControllable& other) = delete;
         RemoteControllable& operator=(const RemoteControllable& other) = delete;
@@ -105,7 +105,7 @@ class RemoteControllable {
          * It might be used in the commands the user has to type, so keep
          * it short
          */
-        virtual std::string get_rc_name() const { return m_name; }
+        virtual std::string get_rc_name() const { return m_rc_name; }
 
         /* Return a list of possible parameters that can be set */
         virtual std::list<std::string> get_supported_parameters() const;
@@ -126,7 +126,7 @@ class RemoteControllable {
         virtual const std::string get_parameter(const std::string& parameter) const = 0;
 
     protected:
-        std::string m_name;
+        std::string m_rc_name;
         std::list< std::vector<std::string> > m_parameters;
 };
 
diff --git a/src/TimestampDecoder.h b/src/TimestampDecoder.h
index 943b241..db8f816 100644
--- a/src/TimestampDecoder.h
+++ b/src/TimestampDecoder.h
@@ -48,7 +48,7 @@ struct frame_timestamp
 
     frame_timestamp() = default;
     frame_timestamp(const frame_timestamp& other) = default;
-    frame_timestamp operator=(const frame_timestamp &rhs)
+    frame_timestamp& operator=(const frame_timestamp &rhs)
     {
         if (this != &rhs) {
             this->timestamp_sec = rhs.timestamp_sec;