Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] New experimental feature: GTK2 basedpacket list(svn rev 2889

From: "Anders Broman" <a.broman@xxxxxxxxx>
Date: Thu, 2 Jul 2009 20:25:39 +0200
Hi
>This goes along the lines of my thinking that we could turn off sorting
>until a capture file is fully loaded.  We could also only turn on sorting
>if the user clicks on one of the column headings by using a number for
>sort_id to signify no sorting.

That seems to work any way, I've checked in the change.
Regards
Anders

-----Ursprungligt meddelande-----
Från: wireshark-dev-bounces@xxxxxxxxxxxxx
[mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] För Stephen Fisher
Skickat: den 2 juli 2009 19:36
Till: Developer support list for Wireshark
Ämne: Re: [Wireshark-dev] New experimental feature: GTK2 basedpacket
list(svn rev 28892)

On Thu, Jul 02, 2009 at 10:13:54AM +0200, Anders Broman wrote:

> Great job! Looks quite good I think. 

Thanks!  I'm going to work on the coloring rules next.

> I played with it a bit and loading large files is quite slow but 
> removing the resorting for every row speeds it up to be faster than 
> the original code I think. A 61Mb trace New packet list with patch 
> ~11s, old WS (SVN Rev 27374) ~16s

Great.

> Memmory usage is much higher though 183 744Kb vs 252 388K. Should some 
> columns be stored as numbers instead of strings?

Probably.

> Index: packet_list_store.c
> ===================================================================
> --- packet_list_store.c (revision 28916)
> +++ packet_list_store.c (working copy)
> @@ -516,7 +516,7 @@
> 
>         gtk_tree_path_free(path);
> 
> -       packet_list_resort(packet_list);
> +       //packet_list_resort(packet_list);
>  }

This goes along the lines of my thinking that we could turn off sorting 
until a capture file is fully loaded.  We could also only turn on 
sorting if the user clicks on one of the column headings by using a 
number for sort_id to signify no sorting.


Steve

___________________________________________________________________________
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