Wireshark · Ethereal-dev: Re: [Ethereal-dev] Autogenerating dissectors ...

[Guy Harris <gharris@xxxxxxxxx>]

On Fri, May 25, 2001 at 06:19:00AM +0900, Richard Sharpe wrote:
> Perhaps I can push code like:
> 
>   if (tree) {
> 
>     # Handle tree view display
> 
>  } else {
> 
>    # Handle summary view display
> 
>  }

Actually, code like that isn't necessarily correct, as a single call to
a dissector may be required to generate both protocol tree information
*and* summary view information - it's more like

	if (check_col(pinfo->fd, COL_PROTOCOL))
		handle summary view information for Protocol column;
	if (check_col(pinfo->fd, COL_INFO))
		handle summary view information for Info column;
	if (tree != NULL) {
		handle tree view display;
	}

Don't forget also that, if the dissector calls any sub-dissectors, it
should do so *regardless* of whether the "tree" argument is null or not.