please sonar: dont apply binary operators to signed ints

[simgrid.git] / examples / msg / dht-kademlia / node.c

diff --git a/examples/msg/dht-kademlia/node.c b/examples/msg/dht-kademlia/node.c
index 99ee294..e854b52 100644 (file)
--- a/examples/msg/dht-kademlia/node.c
+++ b/examples/msg/dht-kademlia/node.c
@@ -115,9 +115,9 @@ unsigned int get_id_in_prefix(unsigned int id, unsigned int prefix)
 {
   if (prefix == 0) {
     return 0;
+  } else {
+    return (1 << ((unsigned int)(prefix - 1))) ^ id;
   }
-  unsigned int n = 1 << (prefix - 1);
-  return n ^ id;
 }
 
 /** @brief Returns the prefix of an identifier.
@@ -127,8 +127,8 @@ unsigned int get_id_in_prefix(unsigned int id, unsigned int prefix)
   */
 unsigned int get_node_prefix(unsigned int id, unsigned int nb_bits)
 {
-  unsigned int j, size = sizeof(unsigned int) * 8;
-  for (j = 0; j < size; j++) {
+  unsigned int size = sizeof(unsigned int) * 8;
+  for (unsigned int j = 0; j < size; j++) {
     if (((id >> (size - 1 - j)) & 0x1) != 0) {
       return nb_bits - (j);
     }