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.
