clean up PR

src/fsfw/datapool/AccessLocalPoolF.h

@@ -1,23 +0,0 @@
-#pragma once
-
-class LocalPoolManager;
-class MutexIF;
-
-/**
- * @brief 	Accessor class which can be used by classes which like to use the pool manager.
- */
-class AccessPoolManagerIF {
- public:
-  virtual ~AccessPoolManagerIF() {};
-
-  virtual MutexIF* getLocalPoolMutex() = 0;
-
-  /**
-   * Can be used to get a handle to the local data pool manager.
-   * This function is protected because it should only be used by the
-   * class imlementing the interface.
-   */
-  virtual LocalPoolManager* getPoolManagerHandle() = 0;
-
- protected:
-};

src/fsfw/datapool/PoolDataSetIF.h

@@ -10,7 +10,7 @@
  */
 class PoolDataSetIF : virtual public DataSetIF, virtual public ReadCommitIF {
  public:
-  virtual ~PoolDataSetIF() {};
+  ~PoolDataSetIF() override = default;
 
   /**
    * @brief   Most underlying data structures will have a pool like structure
@@ -18,17 +18,13 @@ class PoolDataSetIF : virtual public DataSetIF, virtual public ReadCommitIF {
    *          thread-safety
    * @return Lock operation result
    */
-  virtual ReturnValue_t lockDataPool(
-      MutexIF::TimeoutType timeoutType = MutexIF::TimeoutType::WAITING,
-      uint32_t timeoutMs = 20) = 0;
+  virtual ReturnValue_t lockDataPool(MutexIF::TimeoutType timeoutType, uint32_t timeoutMs) = 0;
 
   /**
    * @brief   Unlock call corresponding to the lock call.
    * @return Unlock operation result
    */
   virtual ReturnValue_t unlockDataPool() = 0;
-
-  // virtual bool isValid() const = 0;
 };
 
 #endif /* FSFW_DATAPOOL_POOLDATASETIF_H_ */

src/fsfw/devicehandlers/DeviceHandlerBase.h

@@ -1,5 +1,4 @@
-#ifndef FSFW_DEVICEHANDLERS_DEVICEHANDLERBASE_H_
-#define FSFW_DEVICEHANDLERS_DEVICEHANDLERBASE_H_
+#pragma once
 
 #include <fsfw/housekeeping/PeriodicHkHelper.h>
 
@@ -999,12 +998,6 @@ class DeviceHandlerBase : public DeviceHandlerIF,
    */
   virtual void doOnActivity();
 
-  /**
-   * Required for HasLocalDataPoolIF, return a handle to the local pool manager.
-   * @return
-   */
-  // PeriodicHkGenerationHelper *getHkManagerHandle() override;
-
   const HasHealthIF *getOptHealthIF() const override;
   const HasModesIF &getModeIF() const override;
 
@@ -1396,8 +1389,6 @@ class DeviceHandlerBase : public DeviceHandlerIF,
 
   ReturnValue_t handleDeviceHandlerMessage(CommandMessage *message);
 
-  // virtual dur_millis_t getPeriodicOperationFrequency() const override;
-
   void parseReply(const uint8_t *receivedData, size_t receivedDataLen);
 
   void handleTransitionToOnMode(Mode_t commandedMode, Submode_t commandedSubmode);
@@ -1417,6 +1408,4 @@ class DeviceHandlerBase : public DeviceHandlerIF,
    * @brief Disables all commands and replies when device is set to MODE_OFF
    */
   void disableCommandsAndReplies();
-};
-
-#endif /* FSFW_DEVICEHANDLERS_DEVICEHANDLERBASE_H_ */
+};

src/fsfw/devicehandlers/FreshDeviceHandlerBase.h

@@ -112,10 +112,6 @@ class FreshDeviceHandlerBase : public SystemObject,
    */
   virtual ReturnValue_t handleCommandMessage(CommandMessage* message) = 0;
 
-  // HK manager abstract functions.
-  // ReturnValue_t initializeLocalDataPool(localpool::DataPool& localDataPoolMap,
-  // PeriodicHkGenerationHelper& poolManager) override = 0;
-
   // Mode abstract functions
   ReturnValue_t checkModeCommand(Mode_t mode, Submode_t submode,
                                  uint32_t* msToReachTheMode) override = 0;
@@ -125,7 +121,7 @@ class FreshDeviceHandlerBase : public SystemObject,
   ReturnValue_t executeAction(ActionId_t actionId, MessageQueueId_t commandedBy,
                               const uint8_t* data, size_t size) override = 0;
   // Executable overrides.
-  virtual ReturnValue_t performOperation(uint8_t opCode) override;
+  ReturnValue_t performOperation(uint8_t opCode) override;
 
   /**
    * This calls the FDIR instance event trigger function.