Ethereal-dev: Re: [Ethereal-dev] Latest Ethereal profiling data ...
On Tue, Dec 02, 2003 at 03:20:25PM -0800, Guy Harris wrote:
> Mainly called from "cell_size_request()", which is, in turn, mainly
> called from "gtk_clist_optimal_column_width()", called mainly from
> "gtk_clist_set_column_auto_resize()". Not automatically sizing the
> columns might be the right answer here.
Well, it made about a 10% difference when reading in one test capture
(without a color filter).
I've checked in a change to get rid of the column auto-resizing. A side
effect is that it no longer picks the too-narrow column widths that I've
seen it pick.
This means that the rightmost column (normally the Info column) is, by
default, wide enough to fill up the window but no wider, and you *don't*
get a horizontal scrollbar. You can drag the column title separator
past the right-hand edge of the pane (well, I can, with the
Accelerated-X server and KDE 1.x kwm I have, at least), which will make
the column wide enough to force a scrollbar to appear.
Currently, it doesn't remember the column widths if you change them. We
should do that. I'd suggest doing it in a separate file from the
preferences file, for the reasons mentioned in other messages. (I'd
suggest doing the same with the window size and position.)
It should probably associate column widths with column *types*, not with
columns, e.g. if you set the "Protocol" column to a given width, it
should record that COL_PROTOCOL should have that width - *and*, if you
resize a "command-line specified" time column, it should record that
time columns displaying that particular value should have that width,
e.g. COL_REL_TIME, COL_ABS_TIME, etc., rather than recording that
COL_CLS_TIME should have that width (so that the width can automatically
change if you change the time stamp display format).
