I submitted a change (r44262) that turns on the cross-drawing, It works on this linux machine - I hope to try later on Windows with later versions of gtk/cairo to make sure it looks OK there. I think there are still some quirks, particularly if you mix up pressing the space-bar or toggling using the control window, but it does seem to basically work.
Martin
P.S. I left the calls to cross_draw() (for the 2 methods of turning it on), but left the test inside cross_draw() to avoid it being done twice in the same position without first being cleared.
Is that the patch you checked in? The one problem I've been seeing is that it draws the cross twice when it is first toggled on - then erased when the mouse moved - leaving an unwanted cross...
The problem is that cross_draw() is being called both in toggle_crosshairs() and in callback_cross_on_off(). The fix would probably be to remove the first of those 2 calls.
This code is all a little fragile though - tempted to check in a sanity check in cross_draw() such as:
/* Shouldn't draw twice onto the same position if haven't erased in the
If this is the same/only problem you were seeing, I will check in the first part (i.e. not calling cross_draw() twice), and possibly also the 2nd check (with an error printf() ?).
Sorry Dana I haven't looked at the right-click and drag feature, I will if I find time.
There is a comment in the code to say that the crosshairs with the
lines didn't work well with cairo. The code is still there but things
are currently configured to use the crosshair cursor instead.
I have reused the TCP graph code for another protocol recently, and plan
to try enabling the line-drawing code to see if I can see what goes wrong.
An alternative might be a text control that gets updated with
time+sequence number when the cursor moves. Does anyone know what went
wrong with the cross-hair lines, or on what platforms?
xor does not work with cairo so some
completely different approach is needed.
So
I'm guessing not many people use this feature (assuming it really is a
feature). I've also noticed that the crosshairs have become a small
"+" symbol instead of the vertical and horizontal hairlines that extended
all the way to both axes, which made it easier to determine the values of
the associated coordinates for the crosshair location. It would be
nice if that old behavior returned as well.
Should I be
submitting a bug report/feature request for this
stuff?
> I use the various TCP Graphs quite a bit and just noticed
that scrolling a zoomed TCP Graph, such as a Time Sequence graph, by
clicking and dragging with the right mouse button no longer seems to work
with the new 1.8.0 version. I've tried this both on Windows 7 and Mac
OS X Lion and get the same behavior. Falling back to 1.6.8 restores
the feature. I haven't found this panning feature documented in any of
the Wireshark manuals, so maybe this is not a supported feature, but if it
is expected to work it no longer seems to. > > Has anyone else
encountered this yet? I know I can use the arrow keys, but the
resolution is a bit coarse via that method and it's much less
convenient. > > Thanks in advance! > >
Dana