From 628ce3e7ad35b78d435e530731708ca8961d1e9e Mon Sep 17 00:00:00 2001
From: Julien Danjou <julien@danjou.info>
Date: Sat, 12 Jan 2008 22:00:36 +0100
Subject: [PATCH] use list functions for TagClientLink

---
 config.h |  2 ++
 tag.c    | 37 +++++++------------------------------
 2 files changed, 9 insertions(+), 30 deletions(-)

diff --git a/config.h b/config.h
index b23d4892d..232656573 100644
--- a/config.h
+++ b/config.h
@@ -243,6 +243,8 @@ struct TagClientLink
     TagClientLink *next;
 };
 
+DO_SLIST(TagClientLink, tcl, p_delete);
+
 /** Padding type */
 typedef struct
 {
diff --git a/tag.c b/tag.c
index e9f0276d1..a5ec11a7e 100644
--- a/tag.c
+++ b/tag.c
@@ -32,43 +32,19 @@
 
 extern AwesomeConf globalconf;
 
-static void
-detach_tagclientlink(TagClientLink *tc)
-{
-    TagClientLink *tmp;
-
-    if(globalconf.tclink == tc)
-        globalconf.tclink = tc->next;
-    else
-    {
-        for(tmp = globalconf.tclink; tmp && tmp->next != tc; tmp = tmp->next);
-        tmp->next = tc->next;
-    }
-    
-    p_delete(&tc);
-}
-
 void
 tag_client(Client *c, Tag *t)
 {
-    TagClientLink *tc, *new_tc;
+    TagClientLink *tc;
 
     /* don't tag twice */
     if(is_client_tagged(c, t))
         return;
 
-    new_tc = p_new(TagClientLink, 1);
-
-    if(!globalconf.tclink)
-        globalconf.tclink = new_tc;
-    else
-    {
-        for(tc = globalconf.tclink; tc->next; tc = tc->next);
-        tc->next = new_tc;
-    }
-
-    new_tc->client = c;
-    new_tc->tag = t;
+    tc = p_new(TagClientLink, 1);
+    tc->client = c;
+    tc->tag = t;
+    tcl_list_push(&globalconf.tclink, tc);
 
     widget_invalidate_cache(c->screen, WIDGET_CACHE_CLIENTS);
 }
@@ -81,7 +57,8 @@ untag_client(Client *c, Tag *t)
     for(tc = globalconf.tclink; tc; tc = tc->next)
         if(tc->client == c && tc->tag == t)
         {
-            detach_tagclientlink(tc);
+            tcl_list_detach(&globalconf.tclink, tc);
+            p_delete(&tc);
             break;
         }