- m_list.c: style cleanups

git-svn-id: svn://svn.ircd-hybrid.org/svnroot/ircd-hybrid/branches/8.1.x@2905 82007160-df01-0410-b94d-b575c5fd34c7

1 files changed, 21 insertions, 22 deletions

diff --git a/modules/m_list.c b/modules/m_list.c
index 6415a00..e6c0733 100644
--- a/modules/m_list.c
+++ b/modules/m_list.c
@@ -48,7 +48,7 @@ do_list(struct Client *source_p, int parc, char *parv[])
   struct ListTask *lt = NULL;
   int no_masked_channels = 1;
 
-  if (source_p->localClient->list_task != NULL)
+  if (source_p->localClient->list_task)
   {
     free_list_task(source_p->localClient->list_task, source_p);
     sendto_one(source_p, form_str(RPL_LISTEND), me.name, source_p->name);
@@ -67,21 +67,23 @@ do_list(struct Client *source_p, int parc, char *parv[])
     dlink_list *list = NULL;
     int i = 0, errors = 0;
 
-    for (opt = strtoken(&save, parv[1], ","); opt != NULL;
-         opt = strtoken(&save, NULL, ","))
+    for (opt = strtoken(&save, parv[1], ","); opt;
+         opt = strtoken(&save,    NULL, ","))
     {
       switch (*opt)
       {
-        case '<': if ((i = atoi(opt + 1)) > 0)
-                    lt->users_max = (unsigned int) i - 1;
-                  else
-                    errors = 1;
-                  break;
-        case '>': if ((i = atoi(opt + 1)) >= 0)
-                    lt->users_min = (unsigned int) i + 1;
-                  else
-                    errors = 1;
-                  break;
+        case '<':
+          if ((i = atoi(opt + 1)) > 0)
+            lt->users_max = (unsigned int)i - 1;
+          else
+            errors = 1;
+          break;
+        case '>':
+          if ((i = atoi(opt + 1)) >= 0)
+            lt->users_min = (unsigned int)i + 1;
+          else
+            errors = 1;
+          break;
         case '-': break;
         case 'C':
         case 'c':
@@ -89,13 +91,13 @@ do_list(struct Client *source_p, int parc, char *parv[])
           {
             case '<':
               if ((i = atoi(opt + 1)) >= 0)
-                lt->created_max = (unsigned int) (CurrentTime - 60 * i);
+                lt->created_max = (unsigned int)(CurrentTime - 60 * i);
               else
                 errors = 1;
               break;
             case '>':
               if ((i = atoi(opt + 1)) >= 0)
-                lt->created_min = (unsigned int) (CurrentTime - 60 * i);
+                lt->created_min = (unsigned int)(CurrentTime - 60 * i);
               else
                 errors = 1;
               break;
@@ -111,13 +113,13 @@ do_list(struct Client *source_p, int parc, char *parv[])
           {
             case '<':
               if ((i = atoi(opt + 1)) >= 0)
-                lt->topicts_min = (unsigned int) (CurrentTime - 60 * i);
+                lt->topicts_min = (unsigned int)(CurrentTime - 60 * i);
               else
                 errors = 1;
               break;
             case '>':
               if ((i = atoi(opt + 1)) >= 0)
-                lt->topicts_max = (unsigned int) (CurrentTime - 60 * i);
+                lt->topicts_max = (unsigned int)(CurrentTime - 60 * i);
               else
                 errors = 1;
               break;
@@ -143,12 +145,9 @@ do_list(struct Client *source_p, int parc, char *parv[])
           }
           else if (!IsChanPrefix(*opt))
             errors = 1;
-          if (!errors)
-          {
-            char *s = xstrdup(opt);
 
-            dlinkAdd(s, make_dlink_node(), list);
-          }
+          if (!errors)
+            dlinkAdd(xstrdup(opt), make_dlink_node(), list);
       }
     }