Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-bugs: [Wireshark-bugs] [Bug 6787] Move Y.1711 out of MPLS dissector

Date: Thu, 9 Aug 2012 07:08:10 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6787

--- Comment #23 from FF <francesco.fondelli@xxxxxxxxx> 2012-08-09 07:08:08 PDT ---
(In reply to comment #20)
> (In reply to comment #19)
> > About this line :
> > > 
> > >     functype = tvb_get_guint8(tvb, offset);  // will do bounds error if no data
> > >     col_append_fstr(pinfo->cinfo, COL_INFO, " (Y.1711: %s)",
> > >                     (functype == 0x01) ? "CV" :
> > >                     (functype == 0x02) ? "FDI" :
> > >                     (functype == 0x03) ? "BDI" :
> > >                     (functype == 0x07) ? "FDD" :
> > >                     "reserved/unknown");
> > > 
> > It is not possible to use val_to_str(functype, y1711_function_type_vals,
> > "Reserved/unknown (%d)) ?
> 
> Maybe FF wanted to append text shorter than that in the value-string array ? 

yep, compact form (e.g. "FDI") in cinfo and long format (e.g. 
"FDI (Forward Defect Indicator)") in the proto tree.

thanks
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.