diff options
author | Michael Mann <mmann78@netscape.net> | 2016-07-24 08:53:39 -0400 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-07-25 04:26:50 +0000 |
commit | 1da1f945e2988080add4923dc2021753e3b2f7c1 (patch) | |
tree | 2839b66064e34ba99a6d031778330f20497b5e93 /epan/asn1.c | |
parent | ee7f9c33f63532bc69899a0750177756be53c0c1 (diff) |
Fix checkAPI.pl warnings about printf
Many of the complaints from checkAPI.pl for use of printf are when its embedded
in an #ifdef and checkAPI isn't smart enough to figure that out.
The other (non-ifdef) use is dumping internal structures (which is a type of
debug functionality)
Add a "ws_debug_printf" macro for printf to pacify the warnings.
Change-Id: I63610e1adbbaf2feffb4ec9d4f817247d833f7fd
Reviewed-on: https://code.wireshark.org/review/16623
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/asn1.c')
-rw-r--r-- | epan/asn1.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/epan/asn1.c b/epan/asn1.c index c2f718d732..6581e42da7 100644 --- a/epan/asn1.c +++ b/epan/asn1.c @@ -30,6 +30,7 @@ #include <math.h> #ifdef DEBUG #include <stdio.h> +#include <wsutil/ws_printf.h> /* ws_debug_printf */ #endif #include <epan/packet.h> @@ -272,7 +273,7 @@ double asn1_get_real(const guint8 *real_ptr, gint len) { } val = (double) S * N * pow(2, F) * pow(B, E); #ifdef DEBUG - printf("S = %d, N = %lu, F = %u, B = %u, E = %d -> %f\n", S, N, F, B, E, val); + ws_debug_printf("S = %d, N = %lu, F = %u, B = %u, E = %d -> %f\n", S, N, F, B, E, val); #endif } else if (octet & 0x40) { /* SpecialRealValue */ switch (octet & 0x3F) { |