diff options
author | Michael Mann <mmann78@netscape.net> | 2016-07-25 22:58:09 -0400 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-07-26 04:08:50 +0000 |
commit | 907d49483d24698013fcdaec15b78a149751eecb (patch) | |
tree | 001c978c947dd64932fd2d9749cfc46756ac30dc /epan/value_string.c | |
parent | 6fbaf9be34e8745b2781294afdcf2822528a6349 (diff) |
Fix some of the checkAPIs.pl warnings for g_warning.
1. Create ws_g_warning for legitimate uses of g_warning
2. Use proto_tree_add_debug_text
3. Comment some out
Change-Id: Ida044bf40286b955fdd529c4f9907c8e09b3d7c5
Reviewed-on: https://code.wireshark.org/review/16678
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: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/value_string.c')
-rw-r--r-- | epan/value_string.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/epan/value_string.c b/epan/value_string.c index 5832541743..4a93cf6c43 100644 --- a/epan/value_string.c +++ b/epan/value_string.c @@ -29,6 +29,7 @@ #include "proto.h" #include "to_str.h" #include "value_string.h" +#include <wsutil/ws_printf.h> /* ws_g_warning */ /* REGULAR VALUE STRING */ @@ -464,14 +465,14 @@ _try_val_to_str_ext_init(const guint32 val, value_string_ext *vse) /* XXX: Should check for dups ?? */ if (type == VS_BIN_TREE) { if (prev_value > vs_p[i].value) { - g_warning("Extended value string '%s' forced to fall back to linear search:\n" + ws_g_warning("Extended value string '%s' forced to fall back to linear search:\n" " entry %u, value %u [%#x] < previous entry, value %u [%#x]", vse->_vs_name, i, vs_p[i].value, vs_p[i].value, prev_value, prev_value); type = VS_SEARCH; break; } if (first_value > vs_p[i].value) { - g_warning("Extended value string '%s' forced to fall back to linear search:\n" + ws_g_warning("Extended value string '%s' forced to fall back to linear search:\n" " entry %u, value %u [%#x] < first entry, value %u [%#x]", vse->_vs_name, i, vs_p[i].value, vs_p[i].value, first_value, first_value); type = VS_SEARCH; |