please sonar by removing useless asignments and return statements

[simgrid.git] / src / mc / Client.hpp

diff --git a/src/mc/Client.hpp b/src/mc/Client.hpp
index 69248bc..6de722c 100644 (file)
--- a/src/mc/Client.hpp
+++ b/src/mc/Client.hpp
@@ -7,8 +7,9 @@
 #ifndef SIMGRID_MC_CLIENT_H
 #define SIMGRID_MC_CLIENT_H
 
-#include <cstddef>
+#include "src/internal_config.h"
 
+#include <cstddef>
 #include <memory>
 
 #include <xbt/base.h>
@@ -21,6 +22,10 @@
 namespace simgrid {
 namespace mc {
 
+/** Model-checked-side of the communication protocol
+ *
+ *  Send messages to the model-checker and handles message from it.
+ */
 class XBT_PUBLIC() Client {
 private:
   bool active_ = false;
@@ -28,17 +33,19 @@ private:
   static std::unique_ptr<Client> client_;
 public:
   Client();
-  Client(int fd) : active_(true), channel_(fd) {}
+  explicit Client(int fd) : active_(true), channel_(fd) {}
   void handleMessages();
   Channel const& getChannel() const { return channel_; }
   Channel& getChannel() { return channel_; }
-  void mainLoop(void);
+  void mainLoop();
   void reportAssertionFailure(const char* description = nullptr);
   void ignoreMemory(void* addr, std::size_t size);
   void ignoreHeap(void* addr, std::size_t size);
   void unignoreHeap(void* addr, std::size_t size);
   void declareSymbol(const char *name, int* value);
-  void declareStack(void *stack, size_t size, smx_process_t process, ucontext_t* context);
+#if HAVE_UCONTEXT_H
+  void declareStack(void *stack, size_t size, smx_actor_t process, ucontext_t* context);
+#endif
 
   // Singleton :/
   // TODO, remove the singleton antipattern.