Michael Lum wrote:
-----Original Message----- From: Jeff Morriss [mailto:[EMAIL PROTECTED]
Michael Lum wrote:
I wanted to make the following changes to the MTP3 dissector:
1. Add two hidden uint32 fields for 24bit OPC/DPC This allows display filtering using the complete 24 bit
integer value
as an alternative to something like "mtp3.ansi_opc == 214-170-26".
i.e. "mtp3.24bit_opc == 0x66aad6" or "mtp3.24bit_opc == 6728406"
Why not re-use "mtp3.opc" (normally used for ITU) in ANSI--but hidden?
One other reason is that currently the opc header field definition use a 14 bit MASK.
It's been a while since I actually worked on a dissector, but I think it's okay to have 2 hf_ variables that correspond to the same filter string. E.g. something like:
{ &hf_mtp3_itu_opc,
{ "OPC", "mtp3.opc", FT_UINT32, BASE_DEC, NULL, ITU_OPC_MASK,
"", HFILL }},
{ &hf_mtp3_ansi_opc,
{ "DPC", "mtp3.opc",
FT_UINT32, BASE_NONE, NULL, 0x0,
"", HFILL }},
_______________________________________________ Ethereal-dev mailing list [EMAIL PROTECTED] http://www.ethereal.com/mailman/listinfo/ethereal-dev