Hi,
Excellent work.
I can review it and check it in in a few days (I am a bit out of sync with
cvs doing some h.245 stuff right
now.). Or someone else might check it in before me.
When this patch is in, could you also please look at when one use
FT_RELATIVETIME in advanced
so that instead of specifying jyst the y-values in number of miliseconds, it
woulds actually
print x.y ms on the y-axis?
Example of what I mean, try advanced and create a graph where you specify
any of MAX/MIN/AVG
and a field that is relative time, such as time between request and response
for SMB or NFS.
Currently it is not obvious that the y-scale is in ms.
----- Original Message -----
From: "Ronald Henderson"
Sent: Tuesday, December 31, 2002 5:08 AM
Subject: io_stat.c/filter_prefs.h/filter_prefs.c patch: user friendly filter
button enhancement...
>
> Guy/Ronnie:
>
> I have made the following enhancements to the io_stat GUI based on Guy's
> comments. Please see if you like them for submission...
>
> 1) Added a nummeric label identifier for each trace (Use to identifiy
which
> filter preference display is shown)...
>
> 2) Each Display Filter Dialog now has the following text added to its
frame
> title to identify which trace the filter is associated with: "IO-Stat
> (Filter: n)" where n = 1|2|3|4|5
>
> 3) Once a Filter Display Dialog is selected all other display "Filter
> Buttons" are grey-out (disabled).
>
> 4) If a io_stat Filter Display Dialog is display and one closes the
io_stat
> window the corresponding Filter Display Dialog is closed out.
>
> I have tested these patches for both GTK 1.x and 2.x
>
> ---rwh
>
>
>
>
>
>
> ##################################
>
> > BTW: I am working on closing out any open display filter dialogs if the
> > iostat dialog is closed out...
>
> One possibility would be to redo the way that's done in general, so that
> the dialog is closed when the text widget to which the dialog is
> attached is deleted (as that's the underlying problem). That'd involve
> changing all the other places with "Filter:" buttons, though.
>
> One thing I noticed when looking at that was that, unlike the other
> dialog boxes that have only one "Filter:" button, the "IO-Stat" dialog
> box has multiple buttons. Currently, the "Filter:" buttons don't pop up
> separate dialog boxes, but perhaps they should (right now, if you've
> popped up one dialog, the "Filter:" buttons appear to have no effect -
> the dialog box still applies to the filter string for the first button
> you pressed), although that would require that the different filter
> dialog boxes have titles to let the user know which one applies to which
> filter. Unfortunately, there's no obvious choice of a string to put
> into there, as the 5 filters don't have separate names (and I don't know
> whether any window system on which Ethereal works provides an API for
> setting the color of a title bar, so that you could say "Filter
> expression for [blue bar]").
>
>