tree-wide: use SWAP_TWO a bit more

coccinelle/swap-two.cocci

@@ -0,0 +1,7 @@
+@@
+expression x, y, z;
+@@
+- z = x;
+- x = y;
+- y = z;
++ SWAP_TWO(x, y);

src/basic/prioq.c

@@ -65,9 +65,6 @@ int prioq_ensure_allocated(Prioq **q, compare_func_t compare_func) {
 }
 
 static void swap(Prioq *q, unsigned j, unsigned k) {
-        void *saved_data;
-        unsigned *saved_idx;
-
         assert(q);
         assert(j < q->n_items);
         assert(k < q->n_items);
@@ -75,12 +72,8 @@ static void swap(Prioq *q, unsigned j, unsigned k) {
         assert(!q->items[j].idx || *(q->items[j].idx) == j);
         assert(!q->items[k].idx || *(q->items[k].idx) == k);
 
-        saved_data = q->items[j].data;
-        saved_idx = q->items[j].idx;
-        q->items[j].data = q->items[k].data;
-        q->items[j].idx = q->items[k].idx;
-        q->items[k].data = saved_data;
-        q->items[k].idx = saved_idx;
+        SWAP_TWO(q->items[j].data, q->items[k].data);
+        SWAP_TWO(q->items[j].idx, q->items[k].idx);
 
         if (q->items[j].idx)
                 *q->items[j].idx = j;

src/shared/barrier.c

@@ -151,8 +151,6 @@ void barrier_destroy(Barrier *b) {
  * this is currently not needed so it hasn't been implemented.
  */
 void barrier_set_role(Barrier *b, unsigned role) {
-        int fd;
-
         assert(b);
         assert(IN_SET(role, BARRIER_PARENT, BARRIER_CHILD));
         /* make sure this is only called once */
@@ -164,9 +162,7 @@ void barrier_set_role(Barrier *b, unsigned role) {
                 b->pipe[0] = safe_close(b->pipe[0]);
 
                 /* swap me/them for children */
-                fd = b->me;
-                b->me = b->them;
-                b->them = fd;
+                SWAP_TWO(b->me, b->them);
         }
 }