* Created a typedef for the packet-transfromation-function. It is easier
authorBernd Petrovitsch <bernd@firmix.at>
Wed, 15 Nov 2006 21:13:52 +0000 (21:13 +0000)
committerBernd Petrovitsch <bernd@firmix.at>
Wed, 15 Nov 2006 21:13:52 +0000 (21:13 +0000)
  to parse (at least IMHO) than the "int (*)(char *, int *)" all over
  the place.
* simplified the source in net_outbuffer_bytes_left() without changing
  the semantics. Either this is correct or it should have been different
  before.
* reduced duplicated code in del_ptf()

src/net_olsr.c
src/net_olsr.h

index e6d726a..c804c7c 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: net_olsr.c,v 1.15 2006/11/15 20:58:51 bernd67 Exp $
+ * $Id: net_olsr.c,v 1.16 2006/11/15 21:13:52 bernd67 Exp $
  */
 
 #include "net_olsr.h"
@@ -88,7 +88,7 @@ struct olsr_netbuf
 
 struct ptf
 {
-  int (*function)(char *, int *);
+  packet_transform_function function;
   struct ptf *next;
 };
 
@@ -349,14 +349,10 @@ net_outbuffer_push_reserved(struct interface *ifp, olsr_u8_t *data, olsr_u16_t s
 int
 net_outbuffer_bytes_left(struct interface *ifp)
 {
-  int remaining;
-
   if(!netbufs[ifp->if_nr])
     return 0;
 
-  remaining = netbufs[ifp->if_nr]->maxsize - netbufs[ifp->if_nr]->pending;
-
-  return remaining ? remaining : 0;
+  return netbufs[ifp->if_nr]->maxsize - netbufs[ifp->if_nr]->pending;
 }
 
 
@@ -369,7 +365,7 @@ net_outbuffer_bytes_left(struct interface *ifp)
  * @returns 1
  */
 int
-add_ptf(int (*f)(char *, int *))
+add_ptf(packet_transform_function f)
 {
 
   struct ptf *new_ptf;
@@ -393,7 +389,7 @@ add_ptf(int (*f)(char *, int *))
  *  0 if not
  */
 int
-del_ptf(int (*f)(char *, int *))
+del_ptf(packet_transform_function f)
 {
   struct ptf *tmp_ptf, *prev;
 
@@ -406,15 +402,10 @@ del_ptf(int (*f)(char *, int *))
        {
          /* Remove entry */
          if(prev == NULL)
-           {
              ptf_list = tmp_ptf->next;
-             free(tmp_ptf);
-           }
          else
-           {
              prev->next = tmp_ptf->next;
-             free(tmp_ptf);
-           }
+          free(tmp_ptf);
          return 1;
        }
       prev = tmp_ptf;
index fde1298..1fae727 100644 (file)
@@ -36,7 +36,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: net_olsr.h,v 1.4 2006/11/15 20:58:51 bernd67 Exp $
+ * $Id: net_olsr.h,v 1.5 2006/11/15 21:13:52 bernd67 Exp $
  */
 
 
@@ -49,6 +49,8 @@
 #include <arpa/inet.h>
 #include <net/if.h>
 
+typedef int (*packet_transform_function)(char *, int *);
+
 #ifdef USE_LIBNET
 char *
 get_libnet_errbuf(void);
@@ -106,10 +108,10 @@ char *
 olsr_ip_to_string(union olsr_ip_addr *);
 
 int
-add_ptf(int (*)(char *, int *));
+add_ptf(packet_transform_function);
 
 int
-del_ptf(int (*f)(char *, int *));
+del_ptf(packet_transform_function);
 
 olsr_bool
 olsr_validate_address(union olsr_ip_addr *);