Ethereal-dev: Re: [Ethereal-dev] Ethereal performance, after Guy's changes ...

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

Date Index Thread Index Other Months All Mailing Lists
Date Prev Date Next Thread Prev Thread Next
From: "Lars Ruoff" <lars.ruoff@xxxxxxxxxxxxxxxxxx>
Date: Wed, 19 Nov 2003 09:24:21 +0100
>
> > I don't understand tap very well but rtp is always on ==> tree is
> > never NULL in dissect_packet.
>
> That's a bug.  "register_tap_listener_rtp_stream()" should *NOT* call
> "register_tap_listener()" - that listener should only be registered if
> somebody *requests* that tap.

Well, i asked the question the time i was submitting this, if it would be
better to register the listener by default or register it on demand. There
was no response at that time so i kept the registration on startup which
sames me a lot of time.
Note that registering the tap on demand means that the whole file does have
to get redissected (long on large files) before being able to show the RTP
streams list. (Registering on startup means the RTP streams list is
available right after file import)
Perhaps there could be a user preference option indicating wether the tap
should be registered on startup?

regards,
Lars Ruoff