Wireshark · Ethereal-dev: [Ethereal-dev] Unnecessary checks to see if COL

Ethereal-dev: [Ethereal-dev] Unnecessary checks to see if COL_PROTOCOL is being displayed

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Richard Sharpe <rsharpe@xxxxxxxxxxxxxxxxx>

Date: Thu, 18 Sep 2003 00:16:21 -0700 (PDT)

Hi,

I was documenting some stuff on dissectors when I noticed that pretty much 
all dissectors have this as the first statement in them:

        if (check_col(pinfo->cinfo, COL_PROTOCOL))
                col_set_str(pinfo->cinfo, COL_PROTOCOL, "POP");

However, if it is being called to build the protocol tree, should it 
bother to update the summary pane, as that has most likely already been 
done.

Regards
-----
Richard Sharpe, rsharpe[at]ns.aus.com, rsharpe[at]samba.org, 
sharpe[at]ethereal.com, http://www.richardsharpe.com

Follow-Ups:
- Re: [Ethereal-dev] Unnecessary checks to see if COL_PROTOCOL is being displayed
  - From: Guy Harris

Prev by Date: Re: [Ethereal-dev] artnet plugin seems to need libtool which is causing problems on FreeBSD
Next by Date: [Ethereal-dev] Questions about Reassembly
Previous by thread: Re: [Ethereal-dev] Crash for get_tcp_conversation_data
Next by thread: Re: [Ethereal-dev] Unnecessary checks to see if COL_PROTOCOL is being displayed
Index(es):
- Date
- Thread