diff options
| author | Sylvain Munaut <tnt@246tNt.com> | 2010-02-21 18:23:24 +0100 |
|---|---|---|
| committer | Harald Welte <laforge@gnumonks.org> | 2010-02-23 15:02:41 +0100 |
| commit | 16aeb95d000e8c86e00cb13a9b88a3b1be7792f2 (patch) | |
| tree | a6045064d41b9ea35547f8b73ffc091dd41164ed /src/target/firmware/comm | |
| parent | 2dba741e054fb5cf3198cb9e313abad15e9e36b2 (diff) | |
comm/msgb.c: Style cleanup in initialization
Don't assign the same var twice & init all from the same source.
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
Diffstat (limited to 'src/target/firmware/comm')
| -rw-r--r-- | src/target/firmware/comm/msgb.c | 18 |
1 files changed, 6 insertions, 12 deletions
diff --git a/src/target/firmware/comm/msgb.c b/src/target/firmware/comm/msgb.c index 3bd5bd04..51f00ff5 100644 --- a/src/target/firmware/comm/msgb.c +++ b/src/target/firmware/comm/msgb.c @@ -84,12 +84,10 @@ 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->data = msg->_data; + msg->head = msg->_data; + msg->tail = msg->_data; return msg; } @@ -120,15 +118,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->data = msg->data; - /* reset tail pointer */ - msg->tail = msg->data; + msg->data = msg->_data; + msg->head = msg->_data; + msg->tail = msg->_data; - /* reset pointers */ msg->l2h = NULL; msg->l3h = NULL; } |