Wireshark-bugs: [Wireshark-bugs] [Bug 5858] some 'set but not used' compilation fixes
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Tue, 26 Apr 2011 11:58:37 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5858

Yaniv Kaul <mykaul@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6219|                            |review_for_checkin?
               Flag|                            |

--- Comment #13 from Yaniv Kaul <mykaul@xxxxxxxxx> 2011-04-26 11:58:36 PDT ---
Created an attachment (id=6219)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=6219)
epan/dissectors/packet-bacapp.c patch

This specific patch is a bit more invasive than the rest:
1. Some that were not initially used, I've made useful. There was an unused
tree item, I've made it a subtree.
2. Some variables were set but not used, by a function. Once I commented them
out, the whole function, fGetMaxAPDUSize(), became not used - so I commented it
out as well.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.