Wireshark-bugs: [Wireshark-bugs] [Bug 7936] new dissector for Media Resource Control Protocol Ve

Date: Tue, 13 Nov 2012 11:18:05 -0800 (PST)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7936

--- Comment #7 from Michael Mann <mmann78@xxxxxxxxxxxx> 2012-11-13 11:18:05 PST ---
(In reply to comment #6)
> (In reply to comment #5)
> > Is there a reason for the effectively empty header file?
> There should not be an empty header, ever. Although, I am not sure what exactly
> do you mean.

The patch contains packet-mrcpv2.h (and puts it in the makefile), but there is
(effectively) nothing in the file.  packet-mrcpv.h (which this seems to be
copying) at least has functionality in it to justify its existance.  I don't
think packet-mrcpv2.h (and including it in the makefiles) should be included in
the patch.

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