Re: [Wireshark-dev] proposed graph_analysis.c change

2013-03-23 Thread Jaap Keuter
On 03/23/2013 12:58 AM, Ed Beroset wrote: In working on fixing a bug today, I made a proposed change or two to graph_analysis.c. See https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7418 for the context. The first patch was a very conservative one that simply added a bit of code to

Re: [Wireshark-dev] proposed graph_analysis.c change

2013-03-23 Thread Ed Beroset
Jaap Keuter wrote: On 03/23/2013 12:58 AM, Ed Beroset wrote: In working on fixing a bug today, I made a proposed change or two to graph_analysis.c. See https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7418 for the context. The first patch was a very conservative one that simply added a bit

Re: [Wireshark-dev] proposed graph_analysis.c change

2013-03-23 Thread Guy Harris
On Mar 23, 2013, at 1:00 PM, Ed Beroset bero...@mindspring.com wrote: One thing still puzzling me is the comment in the (now elided) code for pane_callback which says repaint the comment area because when moving the pane position there are times that the expose_event_comments is not called.

[Wireshark-dev] proposed graph_analysis.c change

2013-03-22 Thread Ed Beroset
In working on fixing a bug today, I made a proposed change or two to graph_analysis.c. See https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7418 for the context. The first patch was a very conservative one that simply added a bit of code to address a problem with not being able to resize