msgb.c: Style cleanup in initialization
authorSylvain Munaut <tnt@246tNt.com>
Wed, 24 Feb 2010 21:57:46 +0000 (22:57 +0100)
committerHarald Welte <laforge@gnumonks.org>
Wed, 24 Feb 2010 21:57:46 +0000 (22:57 +0100)
Don't assign the same var twice & init all from the same source.

src/msgb.c

index 2521ca8..60af373 100644 (file)
@@ -44,12 +44,8 @@ struct msgb *msgb_alloc(uint16_t size, const char *name)
        msg->data_len = size;
        msg->len = 0;
        msg->data = msg->_data;
-
-       msg->head = msg->data;
-       msg->data = msg->data;
-       /* reset tail pointer */
-       msg->tail = msg->data;
-       //msg->end = msg->tail + size;
+       msg->head = msg->_data;
+       msg->tail = msg->_data;
 
        return msg;
 }
@@ -79,16 +75,11 @@ struct msgb *msgb_dequeue(struct llist_head *queue)
 
 void msgb_reset(struct msgb *msg)
 {
-       msg->len = 0;
        msg->len = 0;
        msg->data = msg->_data;
+       msg->head = msg->_data;
+       msg->tail = msg->_data;
 
-       msg->head = msg->data;
-       msg->data = msg->data;
-       /* reset tail pointer */
-       msg->tail = msg->data;
-
-       /* reset pointers */
        msg->bts_link = NULL;
        msg->trx = NULL;
        msg->lchan = NULL;