Wireshark-bugs: [Wireshark-bugs] [Bug 6787] Move Y.1711 out of MPLS dissector
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Sat, 28 Jul 2012 08:44:36 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6787

--- Comment #16 from FF <francesco.fondelli@xxxxxxxxx> 2012-07-28 08:44:35 PDT ---
(In reply to comment #14)
> FF:

Hi Bill,

> There are two cases (see below) wherein the dissector does a 'return 0' which
> seem incorrect to me. (The 3rd case of 'return 0' is redundant).

agree

[cut]
> (Not the most earthshaking issue), 

:-)

> but what do you think ? (See below for my suggested changes).
[cut]

agree with your suggestions

I cooked a patch, attachment 8853.

thank you
ciao
FF

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