From a4ca61d8343dcc64fc4667d2ac3ad74045825059 Mon Sep 17 00:00:00 2001
From: Robin Mueller <robin.mueller.m@gmail.com>
Date: Thu, 18 Mar 2021 00:01:29 +0100
Subject: [PATCH] bugfix and printout improvement

---
 datapool/PoolDataSetBase.cpp            | 8 ++++----
 internalError/InternalErrorReporter.cpp | 1 -
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/datapool/PoolDataSetBase.cpp b/datapool/PoolDataSetBase.cpp
index bdca22c3d..cd4eb4265 100644
--- a/datapool/PoolDataSetBase.cpp
+++ b/datapool/PoolDataSetBase.cpp
@@ -61,11 +61,11 @@ ReturnValue_t PoolDataSetBase::read(MutexIF::TimeoutType timeoutType,
     }
     else {
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-        sif::error << "DataSet::read(): Call made in wrong position. Don't forget to commit"
-                " member datasets!" << std::endl;
+        sif::warning << "PoolDataSetBase::read: Call made in wrong position. Don't forget to "
+                "commit member datasets!" << std::endl;
 #else
-        sif::printError("DataSet::read(): Call made in wrong position. Don't forget to commit"
-                " member datasets!\n");
+        sif::printWarning("PoolDataSetBase::read: Call made in wrong position. Don't forget to "
+                "commit member datasets!\n");
 #endif /* FSFW_CPP_OSTREAM_ENABLED == 1 */
         result = SET_WAS_ALREADY_READ;
     }
diff --git a/internalError/InternalErrorReporter.cpp b/internalError/InternalErrorReporter.cpp
index 402071b2a..af9acc0d0 100644
--- a/internalError/InternalErrorReporter.cpp
+++ b/internalError/InternalErrorReporter.cpp
@@ -54,7 +54,6 @@ ReturnValue_t InternalErrorReporter::performOperation(uint8_t opCode) {
 #endif
 
     PoolReadGuard readGuard(&internalErrorDataset);
-    internalErrorDataset.read(timeoutType, timeoutMs);
     if(readGuard.getReadResult() == HasReturnvaluesIF::RETURN_OK) {
         internalErrorDataset.queueHits.value += newQueueHits;
         internalErrorDataset.storeHits.value += newStoreHits;