From 7174762b51375c0c95a7b3dd8108904f9a90afca Mon Sep 17 00:00:00 2001 From: Alexis La Goutte Date: Mon, 4 Jun 2012 22:25:10 +0000 Subject: Fix some Dead Store (Dead assignement/Dead increment) Warning found by Clang svn path=/trunk/; revision=43088 --- plugins/unistim/packet-unistim.c | 60 +++++++++++++++++++++++++++++++++++++++- 1 file changed, 59 insertions(+), 1 deletion(-) (limited to 'plugins/unistim') diff --git a/plugins/unistim/packet-unistim.c b/plugins/unistim/packet-unistim.c index 7991dbee4f..c19031ba27 100644 --- a/plugins/unistim/packet-unistim.c +++ b/plugins/unistim/packet-unistim.c @@ -506,7 +506,10 @@ dissect_unistim_message(proto_tree *unistim_tree,packet_info *pinfo,tvbuff_t *tv offset+=(msg_len-2); } - + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } return offset; } @@ -594,6 +597,11 @@ dissect_basic_phone(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } /*DONE*/ @@ -666,6 +674,11 @@ dissect_basic_switch(proto_tree *msg_tree, } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } @@ -739,6 +752,11 @@ dissect_broadcast_switch(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } /*DONE Haven't seen any phone broadcasts, wouldn't expect to*/ @@ -1382,6 +1400,11 @@ dissect_display_switch(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } /*DONE*/ @@ -1488,6 +1511,11 @@ dissect_display_phone(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } @@ -1601,6 +1629,11 @@ dissect_key_indicator_switch(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } @@ -1690,6 +1723,11 @@ dissect_key_indicator_phone(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } @@ -1829,6 +1867,11 @@ dissect_network_switch(proto_tree *msg_tree, } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } @@ -2007,6 +2050,11 @@ dissect_network_phone(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } /*DONE*/ @@ -2436,6 +2484,11 @@ dissect_audio_switch(proto_tree *msg_tree,packet_info *pinfo, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } @@ -2655,6 +2708,11 @@ dissect_audio_phone(proto_tree *msg_tree, offset+=msg_len; } + if(msg_len){ + /* TODO: add Expert info to indicate there is unknown data ! + For the moment, this code only remove Clang Warnings about not used msg_len... */ + } + return offset; } -- cgit v1.2.3