Comment # 15
on bug 10601
from Gerald Combs
I added the timer to the Qt code under the theory that updating the UI every
few ms would use less CPU under heavy load. If this isn't the case we can
remove it.
It seems odd that we're handling a UI preference in file.c. We have callbacks
that fire when new packets show up: capture_cb_capture_update_continue and
capture_cb_capture_fixed_continue. Would it make more sense to handle automatic
scrolling there?
Note that ui/qt/packet_list.cpp:packet_list_moveto_end has a bug: it selects
the last packet instead of scrolling to the end. It should be a no-op if we
keep the timer or adjust the scrollbar if we don't.
You are receiving this mail because:
- You are watching all bug changes.