Michael Mann
changed
bug 6685
What |
Removed |
Added |
Attachment #11644 Flags |
review_for_checkin?
|
review_for_checkin-
|
Comment # 8
on bug 6685
from Michael Mann
Comment on attachment 11644 [details]
Updated patch for TShark 1.10.2 (SVN Rev 51937 from /releases/wireshark-1.10.2)
The reason I'm rejecting the patch is not the functionality, but the
architecture. Wireshark has always kept TShark and Wireshark stats separately,
but I think there is a desire to change that (with Qt a big reason for that).
You are on the right path creating an additional abstraction layer, but I don't
think that layer should expose "GUI" elements like the GTK (especially with Qt
soon replacing it).
Not sure if you want to wait for a more formal stat API that merges TShark and
(GUI)shark, or if you're brave enough to take it on yourself.
You are receiving this mail because:
- You are watching all bug changes.