Ethereal-dev: Re: [Ethereal-dev] Conversation/Endpoint list performance patch

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

From: Ian Schorr <ethereal@xxxxxxxxxxxxx>
Date: Thu, 15 Jul 2004 14:16:09 -0400
Okay, let's try this again.

Attachment: endpoint_talkers_table.c.diff
Description: Binary data

Attachment: hostlist_table.c.diff
Description: Binary data



On Jul 14, 2004, at 7:01 AM, Ulf Lamping wrote:

Ethereal development <ethereal-dev@xxxxxxxxxxxx> schrieb am 14.07.04 02:48:06:

I've checked in your changes.

Right as you checked this in I'd just built new patches that I believe
are more efficient.  I'm just getting around to sending them in =)

Well, that happens sometimes.


It bothered me that the address fields were being written to with 2
additional gtk_clist_set_text calls *per entry* in the list during each
draw update (every 3 seconds), and that there were a huge number of
updates being drawn that didn't need to be (I think it only makes sense
to update periodically, at the tap draw refresh every 3 seconds).

I implemented the name resolvings in these statistics some time ago, but didn't had performance in mind (so blame it on me :-)


Hopefully these new patches solve all of these things.  They're diffed
against the current version of the files in CVS.

Did you keep in mind, that the address resolvings might change data being displayed? Is this done by the 3 seconds update?


However, when I'm using statistics for longer capture files, Ethereal
crashes reproducible on my Win32 machine.

Before the patches, after the patches, or both?


It crashed a long time before, and still afterwards. This is not related to your patches.

Regards, ULFL

_______________________________________________________
WEB.DE Video-Mail - Sagen Sie mehr mit bewegten Bildern
Informationen unter: http://freemail.web.de/?mc=021199

_______________________________________________
Ethereal-dev mailing list
Ethereal-dev@xxxxxxxxxxxx
http://www.ethereal.com/mailman/listinfo/ethereal-dev