Guy Harris wrote:
>
>
> I never said it wasn't in the current CVS tree.
Ok, your last sentence was not quite clear for me, I should read
carefully the whole thread in the future :-)
> The benefits of auto-resizing are:
>
> 1) the code that sets the sizes doesn't have to know the stuff
> that all the different packet decoders put into the columns -
> but it may be possible to achieve this by centralizing some
> of the stuff that puts stuff into the columns, and possibly
> having the decoders explicitly specify how big they think the
> columns should be.
>
> 2) the columns are adjusted so that none of them are too small
> (cutting off some data) or too big (wasting space) - but the
> source and destination address columns, if they contain
> names, may have stuff at the end of the name that is
> redundant (and, if not, the user could manually resize the
> column), and it may be that, for everything except "Info"
> (which should, by default, get all the space that's left), a
> column that's big enough won't be too big (if no row has
> something in that column that's the maximum size).
>
> 3) if you change the time format to one that's wider, you don't
> then have to manually resize the column to a size that
> Ethereal already knows is appropriate.
>
> I'm not willing to give up 3); I found having to manually resize the
> "Time" column if I used "Display/Options" to select "Time of day"
> unacceptable.
That's fine for me.
Resizing on user actions is ok but auto-resizing while capturing with -S
option without any user actions is a bit confusing. The only moment where
it might be allowed to auto-resize is at the end of the capture session
(when the user presses the stop button). And auto-rezing while reading
a captured file is not necessary (but the appropriate widths may be saved
in order to resize at the end of file).
Laurent.
--
Laurent DENIEL | E-mail: deniel@xxxxxxxxxxx
Paris, FRANCE | laurent.deniel@xxxxxxxxxxxxxxxxxxxxxxxxxxxx
| WWW : http://www.worldnet.fr/~deniel
All above opinions are personal, unless stated otherwise.
