diff options
author | D. Ulis <daulis0@gmail.com> | 2016-05-22 14:46:11 -0400 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2016-05-23 15:33:29 +0000 |
commit | 39c8e7e3136d4afc8f09ed745e7ca32ee7f3861c (patch) | |
tree | 3112063de7200cb48102a827e88684df0d9225ef /ui/qt/conversation_colorize_action.cpp | |
parent | 6308d57444a4b2dfccaf2fbc8de2f010f1fe93c5 (diff) |
Qt: Enable Analyze -> Conversation Filter
Exit early if there is no color rule before setting the parent menu enabled/disabled.
Bug: 12458
Change-Id: I37dc093681d3dc28afb60df7fd45e49ab804b2a1
Reviewed-on: https://code.wireshark.org/review/15538
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Diffstat (limited to 'ui/qt/conversation_colorize_action.cpp')
-rw-r--r-- | ui/qt/conversation_colorize_action.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/qt/conversation_colorize_action.cpp b/ui/qt/conversation_colorize_action.cpp index f3b846efd5..a051f444b2 100644 --- a/ui/qt/conversation_colorize_action.cpp +++ b/ui/qt/conversation_colorize_action.cpp @@ -54,7 +54,7 @@ void ConversationAction::setPacketInfo(struct _packet_info *pinfo) // If we're the "New Coloring Rule" item, enable or disable our parent menu. QMenu *parent_submenu = qobject_cast<QMenu *>(parentWidget()); - if (color_number_ > 0 || !parent_submenu) return; + if (color_number_ < 0 || !parent_submenu) return; parent_submenu->setEnabled(enable); } |