Wireshark-dev: Re: [Wireshark-dev] proto_field_is_referenced

From: Evan Huus <eapache@xxxxxxxxx>
Date: Sat, 23 Nov 2013 14:42:10 -0500
Based on a quick skim it looks like a more powerful version of the
usual if (tree) performance checks. When filtering, tree cannot be
NULL as there must be somewhere for the filtered fields to be
attached, but if you are filtering on HTTP then there is no need to
fill in the Ethernet/TCP/IP fields so they can all be skipped by
checking proto_field_is_referenced.

It doesn't appear to be much used though...

Evan

On Sat, Nov 23, 2013 at 2:23 PM,  <mmann78@xxxxxxxxxxxx> wrote:
> Is proto_field_is_references still a valid function?
>
> In looking at the intent, it either looks completely useless or should be
> used on almost all dissectors.  I was leaning more towards useless because I
> can't think of a scenario where tree (is already) != NULL, yet a dissector
> would be called without having its proto field referenced.
>
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives:    http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
>              mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe