Apply clang-tidy suggestions to MsgLogger and WarnManager (#2823)

* Add misc suggestions from clang-tidy for MsgLogger and WarnManager

* add [[maybe_unused]] to private member variable

* fixed issue with unused variable

* fix bug

1 files changed, 3 insertions, 4 deletions

diff --git a/Source/Utils/MsgLogger/MsgLogger.H b/Source/Utils/MsgLogger/MsgLogger.H
index ca55289b2..d5d7e207c 100644
--- a/Source/Utils/MsgLogger/MsgLogger.H
+++ b/Source/Utils/MsgLogger/MsgLogger.H
@@ -280,10 +280,9 @@ namespace MsgLogger{
 
 #endif
 
-        int m_rank = 0         /*! MPI rank of the current process*/;
-        int m_num_procs = 0    /*! Number of MPI ranks*/;
-        int m_io_rank = 0      /*! Rank of the I/O process*/;
-        bool m_am_i_io = false /*! Flag to store if the process is responsible for I/O*/;
+        const int m_rank       /*! MPI rank of the current process*/;
+        const int m_num_procs  /*! Number of MPI ranks*/;
+        const int m_io_rank    /*! Rank of the I/O process*/;
 
         std::map<Msg, std::int64_t> m_messages /*! This stores a map to associate warning messages with the corresponding counters*/;
     };