ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 5535] Remove batadv dissector

Date: Tue, 4 Jan 2011 08:06:07 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5535

Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jeff.morriss.ws@xxxxxxxxx

--- Comment #3 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2011-01-04 08:06:06 PST ---
(In reply to comment #0)
> The batadv dissector wasn't really welcome by the wireshark developers due to
> the fact that batman-adv is a always changing protocol. This created a lot of
> duplicated code inside the dissector. This patch removes the dissector due to
> the massive amount of changes waiting to be added to it which would create even
> more duplicated code.

Really?  I wasn't aware of that. Looks like all the batadv bugs are closed and
the patches were checked in (well, I'm going by what I can find in my 'bugs'
emails).  Admittedly we may be somewhat slow to check them in...

I also don't quite see why this is leading to duplicated code. Do you mean
duplicated between Wireshark and somewhere else or...?

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