Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Also inline xbt_swag_belongs
[simgrid.git] / src / xbt / swag.c
index eda86e4..9d6c209 100644 (file)
@@ -15,9 +15,6 @@
 #include "xbt/log.h"
 #include "xbt/swag.h"
 
-#define PREV(obj,offset) xbt_swag_getPrev(obj,offset)
-#define NEXT(obj,offset) xbt_swag_getNext(obj,offset)
-
 
 /** Creates a new swag.
  * \param offset where the hookup is located in the structure
@@ -81,8 +78,8 @@ void xbt_swag_insert(void *obj, xbt_swag_t swag)
     return;
   }
 
-  PREV(obj, swag->offset) = swag->tail;
-  NEXT(PREV(obj, swag->offset), swag->offset) = obj;
+  xbt_swag_getPrev(obj, swag->offset) = swag->tail;
+  xbt_swag_getNext(xbt_swag_getPrev(obj, swag->offset), swag->offset) = obj;
 
   swag->tail = obj;
 }
@@ -108,8 +105,8 @@ void xbt_swag_insert_at_head(void *obj, xbt_swag_t swag)
     return;
   }
 
-  NEXT(obj, swag->offset) = swag->head;
-  PREV(NEXT(obj, swag->offset), swag->offset) = obj;
+  xbt_swag_getNext(obj, swag->offset) = swag->head;
+  xbt_swag_getPrev(xbt_swag_getNext(obj, swag->offset), swag->offset) = obj;
 
   swag->head = obj;
 }
@@ -135,8 +132,8 @@ void xbt_swag_insert_at_tail(void *obj, xbt_swag_t swag)
     return;
   }
 
-  PREV(obj, swag->offset) = swag->tail;
-  NEXT(PREV(obj, swag->offset), swag->offset) = obj;
+  xbt_swag_getPrev(obj, swag->offset) = swag->tail;
+  xbt_swag_getNext(xbt_swag_getPrev(obj, swag->offset), swag->offset) = obj;
 
   swag->tail = obj;
 }
@@ -154,28 +151,28 @@ void *xbt_swag_remove(void *obj, xbt_swag_t swag)
 
   if ((!obj) || (!swag))
     return NULL;
-  if(!xbt_swag_belongs(obj, swag)) /* Trying to remove an object that
-                                     was not in this swag */
+  if (!xbt_swag_belongs(obj, swag))     /* Trying to remove an object that
+                                           was not in this swag */
     return NULL;
 
-  if (swag->head == swag->tail) {      /* special case */
-    if (swag->head != obj)     /* Trying to remove an object that was not in this swag */
+  if (swag->head == swag->tail) {       /* special case */
+    if (swag->head != obj)      /* Trying to remove an object that was not in this swag */
       return NULL;
     swag->head = NULL;
     swag->tail = NULL;
-    NEXT(obj, offset) = PREV(obj, offset) = NULL;
-  } else if (obj == swag->head) {      /* It's the head */
-    swag->head = NEXT(obj, offset);
-    PREV(swag->head, offset) = NULL;
-    NEXT(obj, offset) = NULL;
-  } else if (obj == swag->tail) {      /* It's the tail */
-    swag->tail = PREV(obj, offset);
-    NEXT(swag->tail, offset) = NULL;
-    PREV(obj, offset) = NULL;
-  } else {                     /* It's in the middle */
-    NEXT(PREV(obj, offset), offset) = NEXT(obj, offset);
-    PREV(NEXT(obj, offset), offset) = PREV(obj, offset);
-    PREV(obj, offset) = NEXT(obj, offset) = NULL;
+    xbt_swag_getNext(obj, offset) = xbt_swag_getPrev(obj, offset) = NULL;
+  } else if (obj == swag->head) {       /* It's the head */
+    swag->head = xbt_swag_getNext(obj, offset);
+    xbt_swag_getPrev(swag->head, offset) = NULL;
+    xbt_swag_getNext(obj, offset) = NULL;
+  } else if (obj == swag->tail) {       /* It's the tail */
+    swag->tail = xbt_swag_getPrev(obj, offset);
+    xbt_swag_getNext(swag->tail, offset) = NULL;
+    xbt_swag_getPrev(obj, offset) = NULL;
+  } else {                      /* It's in the middle */
+    xbt_swag_getNext(xbt_swag_getPrev(obj, offset), offset) = xbt_swag_getNext(obj, offset);
+    xbt_swag_getPrev(xbt_swag_getNext(obj, offset), offset) = xbt_swag_getPrev(obj, offset);
+    xbt_swag_getPrev(obj, offset) = xbt_swag_getNext(obj, offset) = NULL;
   }
   (swag->count)--;
   return obj;
@@ -195,18 +192,19 @@ void *xbt_swag_extract(xbt_swag_t swag)
 
   obj = swag->head;
 
-  if (swag->head == swag->tail) {      /* special case */
+  if (swag->head == swag->tail) {       /* special case */
     swag->head = swag->tail = NULL;
-    PREV(obj, offset) = NEXT(obj, offset) = NULL;
+    xbt_swag_getPrev(obj, offset) = xbt_swag_getNext(obj, offset) = NULL;
   } else {
-    swag->head = NEXT(obj, offset);
-    PREV(swag->head, offset) = NULL;
-    NEXT(obj, offset) = NULL;
+    swag->head = xbt_swag_getNext(obj, offset);
+    xbt_swag_getPrev(swag->head, offset) = NULL;
+    xbt_swag_getNext(obj, offset) = NULL;
   }
   (swag->count)--;
 
   return obj;
 }
+
 /**
  * \param swag a swag
  * \return the number of objects in \a swag
@@ -216,21 +214,10 @@ int xbt_swag_size(xbt_swag_t swag)
   return (swag->count);
 }
 
-/**
- * \param obj an object
- * \param swag a swag
- * \return 1 if \a obj is in the \a swag and 0 otherwise
- */
-int xbt_swag_belongs(void *obj, xbt_swag_t swag)
-{
-  return ((NEXT(obj, swag->offset)) || (PREV(obj, swag->offset))
-      || (swag->head == obj));
-}
-
 
 #ifdef SIMGRID_TEST
 
-XBT_TEST_SUITE("swag","Swag data container");
+XBT_TEST_SUITE("swag", "Swag data container");
 
 typedef struct {
   s_xbt_swag_hookup_t setA;
@@ -239,22 +226,23 @@ typedef struct {
 } shmurtz, s_shmurtz_t, *shmurtz_t;
 
 
-XBT_TEST_UNIT("basic",test_swag_basic,"Basic usage") {
+XBT_TEST_UNIT("basic", test_swag_basic, "Basic usage")
+{
   shmurtz_t obj1, obj2, obj;
-  xbt_swag_t setA,setB;
+  xbt_swag_t setA, setB;
 
-  obj1 = xbt_new0(s_shmurtz_t,1);
-  obj2 = xbt_new0(s_shmurtz_t,1);
+  obj1 = xbt_new0(s_shmurtz_t, 1);
+  obj2 = xbt_new0(s_shmurtz_t, 1);
 
-  obj1->name="Obj 1";
-  obj2->name="Obj 2";
+  obj1->name = "Obj 1";
+  obj2->name = "Obj 2";
 
   xbt_test_add0("Basic usage");
-  xbt_test_log3("%p %p %ld\n",obj1,&(obj1->setB),
-                (long)((char *)&(obj1->setB) - (char *)obj1));
+  xbt_test_log3("%p %p %ld\n", obj1, &(obj1->setB),
+                (long) ((char *) &(obj1->setB) - (char *) obj1));
 
-  setA = xbt_swag_new(xbt_swag_offset(*obj1,setA));
-  setB = xbt_swag_new(xbt_swag_offset(*obj1,setB));
+  setA = xbt_swag_new(xbt_swag_offset(*obj1, setA));
+  setB = xbt_swag_new(xbt_swag_offset(*obj1, setB));
 
   xbt_swag_insert(obj1, setA);
   xbt_swag_insert(obj1, setB);
@@ -262,24 +250,24 @@ XBT_TEST_UNIT("basic",test_swag_basic,"Basic usage") {
   xbt_swag_insert(obj2, setB);
 
   xbt_swag_remove(obj1, setB);
-  /*  xbt_swag_remove(obj2, setB);*/
+  /*  xbt_swag_remove(obj2, setB); */
 
   xbt_test_add0("Traverse set A");
-  xbt_swag_foreach(obj,setA) {
-    xbt_test_log1("Saw: %s",obj->name);
+  xbt_swag_foreach(obj, setA) {
+    xbt_test_log1("Saw: %s", obj->name);
   }
 
   xbt_test_add0("Traverse set B");
-  xbt_swag_foreach(obj,setB) {
-    xbt_test_log1("Saw: %s",obj->name);
+  xbt_swag_foreach(obj, setB) {
+    xbt_test_log1("Saw: %s", obj->name);
   }
 
   xbt_test_add0("Ensure set content and length");
-  xbt_test_assert(  xbt_swag_belongs(obj1,setA));
-  xbt_test_assert(  xbt_swag_belongs(obj2,setA));
+  xbt_test_assert(xbt_swag_belongs(obj1, setA));
+  xbt_test_assert(xbt_swag_belongs(obj2, setA));
 
-  xbt_test_assert(! xbt_swag_belongs(obj1,setB));
-  xbt_test_assert(  xbt_swag_belongs(obj2,setB));
+  xbt_test_assert(!xbt_swag_belongs(obj1, setB));
+  xbt_test_assert(xbt_swag_belongs(obj2, setB));
 
   xbt_test_assert(xbt_swag_size(setA) == 2);
   xbt_test_assert(xbt_swag_size(setB) == 1);