https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Anders Broman anders.bro...@ericsson.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Anders Broman anders.bro...@ericsson.com changed:
What|Removed |Added
Attachment #5697|review_for_checkin?
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #13 from Allan M. Madsen alla...@madsen.dk 2011-01-20 00:39:16
PST ---
(In reply to comment #12)
Am I right that the first channel is AVDTP(s) and second is AVDTP(d)? This is
kind of rule. Then I check whether it is odd/even.
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #14 from Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com
2011-01-20 02:20:48 PST ---
(In reply to comment #13)
(In reply to comment #12)
Am I right that the first channel is AVDTP(s) and second is AVDTP(d)? This
is
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #15 from Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com
2011-01-20 04:40:13 PST ---
Created an attachment (id=5754)
-- (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=5754)
AVDTP trace Nokia N900 - Nokia BH-501
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #12 from Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com
2011-01-19 15:04:41 PST ---
(In reply to comment #11)
The detecting of the AVDTP channel type from counting the L2CAP channels
established with the AVDTP PSM is
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Allan M. Madsen alla...@madsen.dk changed:
What|Removed |Added
CC|
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com changed:
What|Removed |Added
Attachment #5690|0
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com changed:
What|Removed |Added
Attachment #5696|review_for_checkin?
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com changed:
What|Removed |Added
Attachment #5682|review_for_checkin?
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Kovarththanan Rajaratnam kovarththanan.rajarat...@gmail.com changed:
What|Removed |Added
CC|
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com changed:
What|Removed |Added
Attachment #5668|review_for_checkin?
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #6 from Jaap Keuter jaap.keu...@xs4all.nl 2010-12-29 20:45:07 CET
---
(From update of attachment 5682)
Looking better, still many of the points of comment 2 are still valid.
--
Configure bugmail:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #2 from Jaap Keuter jaap.keu...@xs4all.nl 2010-12-28 10:45:50 CET
---
Quick review :
No use of C++ style comments allowed.
Please remove the debug code.
specific to packet-btavdtp.c:
Replace xx2str() into using value_string
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #3 from Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com
2010-12-28 03:18:52 PST ---
(In reply to comment #2)
Quick review :
No use of C++ style comments allowed.
Please remove the debug code.
specific to
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
--- Comment #4 from Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com
2010-12-28 03:25:03 PST ---
(In reply to comment #3)
(In reply to comment #2)
Quick review :
No use of C++ style comments allowed.
Please remove the debug
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5525
Andrei Emeltchenko andrei.emeltchenko.n...@gmail.com changed:
What|Removed |Added
Attachment #5667|0
17 matches
Mail list logo
