[bugzilla-daemon@xxxxxxxxxxxxx]

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2581


Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |martin.r.mathieson@googlemai
                   |                            |l.com




--- Comment #4 from Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>  2008-06-20 09:26:00 PDT ---
I like this idea a lot, but I've noted some problems while playing with the
current patch.

There are lots of places this callback function should be called, or rather it
should be a replacement for the function used everywhere else.


Rather than showing the current position in longer list of filter fields, might
it be possible to restrict the list to those filters matching the current
prefix?

I tried pressing down, hoping that the first visible list item would get
selected, but it didn't.

This popup list doesn't appear when you delete characters back to the last dot.

Once I clicked on a choice, it was selected immediately.  I'd have preferred it
to wait until I've pressed enter.

This is just my opinion, but the ideal behaviour for me would be totally
keyboard-driver, so I could press down to browse them, enter to select one,
escape to dismiss the list and return control to the text control.




I did see a couple of crashes while running under gdb:

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 1111388256 (LWP 8242)]
filter_te_focus_out_cb (filter_te=0x8a900b0, event=0x8342db8, data=0x8ab0ce0)
at filter_dlg.c:1404
1404      if(GTK_IS_WIDGET(data)) {
(gdb) bt
#0  filter_te_focus_out_cb (filter_te=0x8a900b0, event=0x8342db8,
data=0x8ab0ce0) at filter_dlg.c:1404
#1  0x41b59bee in gtk_marshal_BOOLEAN__VOID () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#2  0x41e878bd in g_closure_invoke () from /opt/gnome/lib/libgobject-2.0.so.0
#3  0x41e98243 in g_signal_connect_closure_by_id () from
/opt/gnome/lib/libgobject-2.0.so.0
#4  0x41e9988f in g_signal_emit_valist () from
/opt/gnome/lib/libgobject-2.0.so.0
#5  0x41e99c95 in g_signal_emit () from /opt/gnome/lib/libgobject-2.0.so.0
#6  0x41c448d8 in gtk_widget_get_default_style () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#7  0x41c505b2 in gtk_window_set_auto_startup_notification () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#8  0x08342db8 in ?? ()
#9  0x00000000 in ?? ()
(gdb)      


A second crash looked like this:

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 1111388256 (LWP 8369)]
0x41ea0479 in g_type_check_instance_is_a () from
/opt/gnome/lib/libgobject-2.0.so.0
(gdb) bt
#0  0x41ea0479 in g_type_check_instance_is_a () from
/opt/gnome/lib/libgobject-2.0.so.0
#1  0x08084651 in filter_string_te_key_pressed_cb (filter_te=0x8a8cc90,
event=0x8342d38) at filter_dlg.c:1513
#2  0x41b59bee in gtk_marshal_BOOLEAN__VOID () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#3  0x41e878bd in g_closure_invoke () from /opt/gnome/lib/libgobject-2.0.so.0
#4  0x41e98243 in g_signal_connect_closure_by_id () from
/opt/gnome/lib/libgobject-2.0.so.0
#5  0x41e9988f in g_signal_emit_valist () from
/opt/gnome/lib/libgobject-2.0.so.0
#6  0x41e99c95 in g_signal_emit () from /opt/gnome/lib/libgobject-2.0.so.0
#7  0x41c448d8 in gtk_widget_get_default_style () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#8  0x41c53bb7 in gtk_window_propagate_key_event () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#9  0x41c56a0c in gtk_window_activate_key () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#10 0x41b59bee in gtk_marshal_BOOLEAN__VOID () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#11 0x41e860c7 in g_value_set_static_boxed () from
/opt/gnome/lib/libgobject-2.0.so.0
#12 0x41e878bd in g_closure_invoke () from /opt/gnome/lib/libgobject-2.0.so.0
#13 0x41e98893 in g_signal_connect_closure_by_id () from
/opt/gnome/lib/libgobject-2.0.so.0
#14 0x41e9988f in g_signal_emit_valist () from
/opt/gnome/lib/libgobject-2.0.so.0
#15 0x41e99c95 in g_signal_emit () from /opt/gnome/lib/libgobject-2.0.so.0
#16 0x41c448d8 in gtk_widget_get_default_style () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#17 0x41b5363a in gtk_propagate_event () from
/opt/gnome/lib/libgtk-x11-2.0.so.0
#18 0x41b54857 in gtk_main_do_event () from /opt/gnome/lib/libgtk-x11-2.0.so.0
#19 0x41d5b58a in gdk_add_client_message_filter () from
/opt/gnome/lib/libgdk-x11-2.0.so.0
#20 0x42158abd in g_main_context_dispatch () from
/opt/gnome/lib/libglib-2.0.so.0
#21 0x4215bcbf in g_main_context_check () from /opt/gnome/lib/libglib-2.0.so.0
#22 0x4215c069 in g_main_loop_run () from /opt/gnome/lib/libglib-2.0.so.0
#23 0x41b54cd4 in gtk_main () from /opt/gnome/lib/libgtk-x11-2.0.so.0
#24 0x0808a8fa in main (argc=0, argv=0xbf8a1c78) at main.c:2584
(gdb)                                   


-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.