diff options
author | Guy Harris <guy@alum.mit.edu> | 2019-11-19 13:21:01 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2019-11-19 21:22:34 +0000 |
commit | fc18fbd522bf32d0f86434eee910ff4b4204fa13 (patch) | |
tree | 25b73c8b9452ca1ab6335073e4900b4996b582f8 | |
parent | 193150fdb728cd70a0ceb8e48af259575a0b2a7b (diff) |
Raise a question in a comment.
In PacketList::selectedRow, will we ever have a current index but not a
selection model?
Change-Id: Ie4a4a78b836876d6a890b2a0a5e5cba2de02a870
Reviewed-on: https://code.wireshark.org/review/35140
Reviewed-by: Guy Harris <guy@alum.mit.edu>
-rw-r--r-- | ui/qt/packet_list.cpp | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/ui/qt/packet_list.cpp b/ui/qt/packet_list.cpp index bb91850f37..0b4f2cfa28 100644 --- a/ui/qt/packet_list.cpp +++ b/ui/qt/packet_list.cpp @@ -415,6 +415,10 @@ QList<int> PacketList::selectedRows(bool useFrameNum) } else if (currentIndex().isValid()) { + // + // XXX - will we ever have a current index but not a selection + // model? + // if (! useFrameNum) rows << currentIndex().row(); else |