Wireshark-bugs: [Wireshark-bugs] [Bug 3393] New: Build-up of signals in User Preferences window
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: bugzilla-daemon@xxxxxxxxxxxxx
Date: Mon, 6 Apr 2009 03:35:40 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3393

           Summary: Build-up of signals in User Preferences window
           Product: Wireshark
           Version: 1.0.6
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: Trivial
          Priority: Low
         Component: Wireshark
        AssignedTo: wireshark-bugs@xxxxxxxxxxxxx
        ReportedBy: spoyarek@xxxxxxxxxx


Created an attachment (id=2912)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=2912)
Patch to register signal only once and then deactivate the signal whenever we
change selection and activate again once we're done

Build Information:
wireshark 1.0.6

Copyright 1998-2009 Gerald Combs <gerald@xxxxxxxxxxxxx> and contributors.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Compiled with GTK+ 2.14.7, with GLib 2.18.4, with libpcap 0.9.8, with libz
1.2.3, with POSIX capabilities (Linux), without libpcre, without SMI, without
ADNS, without Lua, with GnuTLS 2.4.2, with Gcrypt 1.4.4, with MIT Kerberos,
without PortAudio, without AirPcap.
NOTE: this build doesn't support the "matches" operator for Wireshark filter
syntax.

Running on Linux 2.6.27.21-170.2.56.fc10.x86_64, with libpcap version 0.9.8.

Built using gcc 4.3.2 20081105 (Red Hat 4.3.2-7).
--
In Edit > Preferences > User Interface > Columns, a new signal handler is added
every time one clicks on the list of columns. Due to this, if someone changes
the tree selection n times, the signal handler is called n+1 times for the
n+1th selection change. I've included a patch to register only one signal
handler and then disable firing of the text entry "changed" whenever a
selection is changed since that is simply not required.

It could be seen as a clean-up patch of sorts, it really doesn't affect
functionality.


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