diff options
author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2015-09-04 13:55:33 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2015-09-04 14:36:49 +0000 |
commit | ad8f8597efb69cf889b3541b0052467888013045 (patch) | |
tree | 4aefdbed46170d2710621542f9152c931a0615f5 | |
parent | ecc51f3ea6c94bffc641691cc907ee1762593ee1 (diff) |
expert_comp_table (GTK): fix will never be executed [-Wunreachable-code]
use #if 0 /#endif for disable code
Found by Clang 3.7
Change-Id: Ie1b6b0936df0ae422baa9b3bb9692f136eb177b1
Reviewed-on: https://code.wireshark.org/review/10380
Petri-Dish: Alexis La Goutte <alexis.lagoutte@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Alexis La Goutte <alexis.lagoutte@gmail.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
-rw-r--r-- | ui/gtk/expert_comp_table.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ui/gtk/expert_comp_table.c b/ui/gtk/expert_comp_table.c index 7fb49d8c5b..5296fa8ffc 100644 --- a/ui/gtk/expert_comp_table.c +++ b/ui/gtk/expert_comp_table.c @@ -204,13 +204,15 @@ error_select_filter_cb(GtkWidget *widget _U_, gpointer callback_data, guint call if (action != ACTION_WEB_LOOKUP && action != ACTION_COPY) { char *msg; - if (0 /*procedure->fvalue_value==NULL*/) { +#if 0 + if (procedure->fvalue_value==NULL) { if (action != ACTION_FIND_FRAME && action != ACTION_FIND_NEXT && action != ACTION_FIND_PREVIOUS) { simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "Wireshark cannot create a filter on this item - %s, try using find instead.", procedure->entries[1]); return; } } +#endif msg = (char *)g_malloc(escape_string_len(procedure->entries[1])); escape_string(msg, procedure->entries[1]); switch(type){ |