little cleanup
authorChristian Thaeter <ct@pipapo.org>
Mon, 8 Oct 2007 17:21:45 +0000 (19:21 +0200)
committerChristian Thaeter <ct@pipapo.org>
Mon, 8 Oct 2007 17:21:45 +0000 (19:21 +0200)
rxpd.c

diff --git a/rxpd.c b/rxpd.c
index c607681..a1afb7e 100644 (file)
--- a/rxpd.c
+++ b/rxpd.c
@@ -617,16 +617,6 @@ rxpd_connection_APPEND_PREPEND_helper (int fd, short event, void* ptr, int do_ap
 {
   struct rxpd_connection* self = (struct rxpd_connection*) ptr;
 
-  if (!event)
-    {
-      if (!self->file)
-        {
-          rxpd_buffer_printf (&self->out, "#ERROR: no such file\n");
-          close (fd);
-          return;
-        }
-    }
-
   if (event == EV_READ)
     {
       int again = -1;
@@ -643,15 +633,21 @@ rxpd_connection_APPEND_PREPEND_helper (int fd, short event, void* ptr, int do_ap
 
               llist_insert_tail (&self->tmp_list, &rule->node);
             }
-          else goto finish;
+          else goto finish;     /* move along, look elsewhere! This goto is not harmful and saves some code. */
         }
     }
+  else if (!event && !self->file)
+    {
+      rxpd_buffer_printf (&self->out, "#ERROR: no such file\n");
+      close (fd);
+      return;
+    }
 
   if (rxpd_buffer_state (&self->in) == RXPD_OK)
     rxpd_connection_schedule (self);
   else
     {
-      // TODO should also print error when any rule compilation failed
+      // TODO should also print error when any rule compilation failed, use tmp_str to save case?
       if (rxpd_buffer_state (&self->in) == RXPD_ERROR)
         rxpd_buffer_printf (&self->out, "#ERROR:\n");
       else