diff options
| author | Guy Harris <guy@alum.mit.edu> | 2011-11-15 05:25:17 +0000 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2011-11-15 05:25:17 +0000 |
| commit | c8283d1291af7d5b5793274d3947d7866d9a9f5c (patch) | |
| tree | a49d9433cb61d4a989416bd6279660190a762d8b /epan/dissectors/packet-reload.c | |
| parent | 0c530d222a1577dffe9789863417614bf01e5ac5 (diff) | |
Rewrite some odd code. Clang Cat does not want:
packet-reload.c:2875:13: warning: Although the value stored to
'local_offset' is used in the enclosing expression, the value is
never actually read from 'local_offset'
although as I read the C90 spec the code is doing pretty much what it
should be doing and the rewritten code does the same thing. However,
it's also a bit more complicated and harder to read than the rewritten
code.
svn path=/trunk/; revision=39840
Diffstat (limited to 'epan/dissectors/packet-reload.c')
| -rw-r--r-- | epan/dissectors/packet-reload.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/epan/dissectors/packet-reload.c b/epan/dissectors/packet-reload.c index d812cb6a89..e38d9f6e04 100644 --- a/epan/dissectors/packet-reload.c +++ b/epan/dissectors/packet-reload.c @@ -2871,8 +2871,7 @@ static int dissect_extensiveroutingmodeoption(tvbuff_t *tvb, packet_info *pinfo, local_offset +=1; dissect_destination_list(tvb, pinfo, destination_tree, offset+local_offset, destination_length, &nDestinations); proto_item_append_text(ti_destination, ": %d elements", nDestinations); - - return local_offset += destination_length; + local_offset += destination_length; } return local_offset; } |