Wireshark-bugs: [Wireshark-bugs] [Bug 6430] BRP Protocol Preference Addition

Date: Wed, 5 Oct 2011 05:17:17 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6430

--- Comment #5 from Rob Kruciak <rob.kruciak@xxxxxx> 2011-10-05 05:17:16 PDT ---
Hello Jaap,

I had seen this method on other protocols prior to submission. Thus, this is
the reason I submitted it in this manner. It actually works very well based on
my testing. To be clear on your comments, in order for a change similar to this
to be accepted, I need to submit changes for BRP and the other protocol that
uses this UDP port and make them both heuristic?

Thanks,

Rob

(In reply to comment #4)
> (From update of attachment 7156 [details])
> We don't want the config.nmake change.
> We don't need yet another way to enable/disable dissectors. This way is not
> even guaranteed to work. If there are conflicting dissectors on the same entry
> of a subdissector table (udp.port in this case) they are to be made heuristic.
> We don't need the whitespace changes. We do need consisted indentation.

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