Wireshark-dev: Re: [Wireshark-dev] [Wireshark-bugs] [Bug 6904] Extensions to wireshark to suppo
From: Krishnamurthy Mayya <krishnamurthymayya@xxxxxxxxx>
Date: Sun, 18 Mar 2012 13:41:30 +0530
Ok..sure..i will follow that approach and submit the dissectors for MPLS-TP Lockinstruct and loopback (RFC 6435 ) and MPLS-TP Fault Management (RFC 6427).
Along with these, i was about coding PSC-State control also(RFC 6378) along with these. But, you have already done it i guess. So i will do rest 2 and submit the patches soon.
No problem right??

On Sun, Mar 18, 2012 at 1:08 AM, <bugzilla-daemon@xxxxxxxxxxxxx> wrote:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6904

FF <francesco.fondelli@xxxxxxxxx> changed:

          What    |Removed                     |Added
----------------------------------------------------------------------------
                CC|                            |francesco.fondelli@xxxxxxxx
                  |                            |m

--- Comment #9 from FF <francesco.fondelli@xxxxxxxxx> 2012-03-17 12:38:04 PDT ---
(In reply to comment #8)
> (In reply to comment #7)
> > Hi Anders,
> >   Just a small doubt. Using "proto_add_item()" has any advantages ??
> > I am just used to proto_tree_add_uint(). All i wanted was a tree by name
> > 'MPLS-TP Lock Instruct message' which when opened gives the relevant fields
> > under it.
> > Actually, for MEP TLV decoding also earlier, I had used the same format. So,
> > was just trying to understand if it could make any difference??
>
> No need local variable (li_version, refresh_timer...)
>
> Also please start your hf by hf_mpls_li...
> About change in bfd dissector, what it this type variable ?
>
> And if it is possible attach a simple patch with all change (from the 3 files)

Hi Alexis, all,

 please do not check this in as it is.  I'm trying to move OAM stuff *out* of
packet-mpls.c.  As soon as possible I'd like to use a new table indexed by
channel_type and remove that ugly if-else-if-else in packet-mpls.c...

 Krishnamurthy Mayya please follow packet-mpls-pm.c or packet-mpls-y1711.c or
packet-mpls-psc.c (bug #6966) approach.  I believe it's easy, elegant
and simple.

hope this make sense and helps
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.
___________________________________________________________________________
Sent via:    Wireshark-bugs mailing list <wireshark-bugs@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://wireshark.org/mailman/options/wireshark-bugs
            mailto:wireshark-bugs-request@xxxxxxxxxxxxx?subject=unsubscribe