please sonar

- don't use void as parameter
- prefer if to ternary conditional

diff --git a/include/simgrid_config.h.in b/include/simgrid_config.h.in
index 0d0cd0d..f27c4ac 100644 (file)
--- a/include/simgrid_config.h.in
+++ b/include/simgrid_config.h.in
@@ -23,7 +23,7 @@ SG_BEGIN_DECL()
 XBT_PUBLIC(void) sg_version_get(int *major,int *minor,int *patch);
 
 /** Display the version information and some additional blurb. */
-XBT_PUBLIC(void) sg_version(void);
+XBT_PUBLIC(void) sg_version();
 SG_END_DECL()
 
 

diff --git a/teshsuite/simdag/is-router/is-router.cpp b/teshsuite/simdag/is-router/is-router.cpp
index 9c9ef7e..eaa9d45 100644 (file)
--- a/teshsuite/simdag/is-router/is-router.cpp
+++ b/teshsuite/simdag/is-router/is-router.cpp
@@ -29,8 +29,14 @@ int main(int argc, char **argv)
   sg_host_t host;
   xbt_dynar_foreach(hosts, it, host) {
     simgrid::kernel::routing::NetPoint* nc = host->pimpl_netpoint;
-    std::printf("   - Seen: \"%s\". Type: %s\n", host->cname(),
-                nc->isRouter() ? "router" : (nc->isNetZone() ? "netzone" : (nc->isHost() ? "host" : "buggy")));
+    const char *type = "buggy";
+    if (nc->isRouter())
+      type = "router";
+    if (nc->isNetZone())
+      type = "netzone";
+    if (nc->isHost())
+      type = "host";
+    std::printf("   - Seen: \"%s\". Type: %s\n", host->cname(), type);
   }
   xbt_dynar_free(&hosts);