Wireshark-bugs: [Wireshark-bugs] [Bug 7675] Bluetooth: Add AVCTP dissector
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Thu, 30 Aug 2012 07:45:12 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7675

--- Comment #9 from Bill Meier <wmeier@xxxxxxxxxxx> 2012-08-30 07:45:11 PDT ---
Two quite minor comments:


1. FALSE should be ENC_NA in the following:


+    ti = proto_tree_add_item(tree, proto_btavctp, tvb, offset, -1, FALSE);
+

2.  As far as I know, wrapping the call to proto_..._register...() with the
    'proto... == -1' test is not required.

+   if (proto_btavctp == -1) {

    Or: is this related to something about "dual-use" (regular or plugin) ?

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