Put printf into a separate "termoutput" API group. For most files,
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Tue, 1 Jul 2008 21:13:32 +0000 (21:13 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Tue, 1 Jul 2008 21:13:32 +0000 (21:13 +0000)
check for it - but not for TShark plugins, as they are expected to
print to the standard output.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@25653 f5534014-38df-0310-8fa8-9805f1628bb7

40 files changed:
codecs/Makefile.nmake
epan/Makefile.nmake
epan/crypt/Makefile.nmake
epan/dfilter/Makefile.nmake
epan/dissectors/Makefile.nmake
epan/ftypes/Makefile.nmake
epan/wslua/Makefile.nmake
plugins/agentx/Makefile.nmake
plugins/artnet/Makefile.nmake
plugins/asn1/Makefile.nmake
plugins/ciscosm/Makefile.nmake
plugins/docsis/Makefile.nmake
plugins/easy_codec/Makefile.nmake
plugins/enttec/Makefile.nmake
plugins/ethercat/Makefile.nmake
plugins/giop/Makefile.nmake
plugins/gryphon/Makefile.nmake
plugins/infiniband/Makefile.nmake
plugins/irda/Makefile.nmake
plugins/lwres/Makefile.nmake
plugins/m2m/Makefile.nmake
plugins/mate/Makefile.nmake
plugins/opcua/Makefile.nmake
plugins/opsi/Makefile.nmake
plugins/pcli/Makefile.nmake
plugins/profinet/Makefile.nmake
plugins/rlm/Makefile.nmake
plugins/rtnet/Makefile.nmake
plugins/rudp/Makefile.nmake
plugins/sbus/Makefile.nmake
plugins/sercosiii/Makefile.nmake
plugins/stats_tree/Makefile.nmake
plugins/tpg/Makefile.nmake
plugins/unistim/Makefile.nmake
plugins/v5ua/Makefile.nmake
plugins/wimax/Makefile.nmake
plugins/wimaxasncp/Makefile.nmake
tools/checkAPIs.pl
wiretap/Makefile.nmake
wsutil/Makefile.nmake

index a255a3f6f84ad5eb88f27871c105843a4916ae6e..093eb03ae2dd7f8d88109a30be502b698ec0fe43 100644 (file)
@@ -35,6 +35,6 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi: 
-       $(PERL) ../tools/checkAPIs.pl \
+       $(PERL) ../tools/checkAPIs.pl -g abort -g termoutput \
        G711a/G711adecode.c \
        G711u/G711udecode.c
index 0dc7c3b7e45e40f16509edb720e71bc62065f1aa..5287dcc37dc80a6bb6c1230b30e7c361ca1de737 100644 (file)
@@ -310,4 +310,4 @@ dtd_grammar.c: $(LEMON)\lemon.exe $(LEMON)\lempar.c dtd_grammar.lemon
        $(LEMON)\lemon t=$(LEMON)\lempar.c dtd_grammar.lemon
 
 checkapi:
-       $(PERL) ../tools/checkAPIs.pl $(LIBWIRESHARK_SRC) \
\ No newline at end of file
+       $(PERL) ../tools/checkAPIs.pl -g termoutput $(LIBWIRESHARK_SRC) \
index 2791340bc7091ad5decaeb9b291750878ce8335d..b05773a2cf696cbc9dfe878074b0e734c65d2f05 100644 (file)
@@ -26,5 +26,5 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl \
+       $(PERL) ../../tools/checkAPIs.pl -g termoutput \
        $(LIBAIRPDCAP_SRC)
index e1942ea3adf79c4889ad23ffb6f2f704b1a11708..c0b28c83afedde81e4646e544ae2ac6a0c9e9c79 100644 (file)
@@ -77,7 +77,7 @@ $(LEMON)\lemon.exe:
        cd ../../epan/dfilter
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl \
+       $(PERL) ../../tools/checkAPIs.pl -g termoutput \
        scanner.l \
        grammar.lemon           \
        dfilter.c               \
index 78c517492503dd518e07f3ecde1ac67960cdf5eb..9d8b06a0085437941833c5d8a543469d9a195d50 100644 (file)
@@ -111,6 +111,6 @@ maintainer-clean: distclean
        rm -f $(GENERATED_FILES)
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort \
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput \
        $(ALL_DISSECTORS_SRC) \
        packet-dcerpc-nt.c
index 848159e79588a9a385286b393c134fb2cfbe733a..5cd5571b6f33156e01169bbb91dfd44296144a8d 100644 (file)
@@ -41,7 +41,7 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl \
+       $(PERL) ../../tools/checkAPIs.pl -g termoutput \
        ftypes.c                \
        ftype-bytes.c           \
        ftype-double.c  \
index d0aeb794d38f7026bf7eedc523cf386bc8fecc84..1631937f90fecbe7844ef2be05ac573aad1e5f48 100644 (file)
@@ -82,7 +82,7 @@ init.lua: template-init.lua make-init-lua.pl ../ftypes/ftypes.h ../../wiretap/wt
        $(PERL) make-init-lua.pl "." template-init.lua > init.lua
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl \
+       $(PERL) ../../tools/checkAPIs.pl -g termoutput \
        $(MODULES)      \
        init_wslua.c \
        register_wslua.c \
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dfd5812d2945622b762c8ced0eddac56f3197adc..6bdc36773d1afbaa7093ea8862e7378dfecd9e2e 100644 (file)
@@ -101,4 +101,4 @@ maintainer-clean: distclean
 
 checkapi:
 # TODO: Fix api's :)
-#      $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+#      $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index b6b9978e9bf129d3632c3c1b088fead1c2cda7f7..3d04e32b3433a487027b8388b395020de51da1cd 100644 (file)
@@ -64,4 +64,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index 0b09e71e517a02808d730c8a06e84d5d9320b147..810e87397e9727c0cfcc542d024a3fb697ed8738 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index 0c7cd6b51487b7c4447af6a961a1fb2a671d9c51..11cd17094fbced0facec5058e09b10e25b9c0cf8 100644 (file)
@@ -51,7 +51,7 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort \
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput \
        packet-cosnaming.c \
        packet-coseventcomm.c \
        packet-tango.c \
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index 590652a0232e67e227880b5ecef1a2aa81b83083..40f8f82f7fc1dd9c3b91310120a42926a46292b1 100644 (file)
@@ -96,4 +96,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index 3c6db870c5610d6ea480b8122b897e35630be3b7..ee662b26ab406c58e8e79988295fb57661c56118 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index 906b718202796027b2c0574fd146ddb1b5c3d513..fdce120155b8c87cbaeb6a6db8de0ee5826e7847 100644 (file)
@@ -128,4 +128,4 @@ $(LEMON)\lemon.exe:
        cd ../../plugins/mate
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index e3cc88ecf94a2e57c621bb40947e24ba5e14effd..89fc5c9376e67622362de7a21158af5863ac653f 100644 (file)
@@ -54,4 +54,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(TAP_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(TAP_SRC)
index 1369afce13807e773f1980c286364d43d3d9477b..d9112f14e3e51022998fe1ed73141d3cf61468ad 100644 (file)
@@ -65,4 +65,4 @@ $(LEMON)\lemon.exe:
        cd ../../plugins/mate
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index dcf18805e80af42c69b4dd3bfa2c3595aa1bc4d8..3cf4dc5b3e26a4f8611a22136be14f951e9f9075 100644 (file)
@@ -100,4 +100,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index 29649a676f43f6d63a2cc94a643b11c6b8275d40..37220cbb5ca22f27dd9ca8427f9a5cbeb8d679b3 100644 (file)
@@ -100,6 +100,6 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort \
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput \
        $(DISSECTOR_SRC) \
        $(DISSECTOR_SUPPORT_SRC)
index 83ce3cf5397a80350176520adc653c57dc16c015..dc3f07e5f66fd12d263ecf918993cbebbc101a0f 100644 (file)
@@ -108,4 +108,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../../tools/checkAPIs.pl -g abort $(DISSECTOR_SRC)
+       $(PERL) ../../tools/checkAPIs.pl -g abort -g termoutput $(DISSECTOR_SRC)
index d6d90d171356afe3f9f7cf6986d464d65f9704d1..61ecef31665f3409dc30bc309f0f1c903051e361 100755 (executable)
@@ -131,9 +131,8 @@ my %APIs = (
                'g_scanner_foreach_symbol',
                'g_scanner_freeze_symbol_table',
                'g_scanner_thaw_symbol_table',
-               # Wireshark should not write to stdout (?)
-               # (Of course tshark should!)
-               'printf',
+               # Use strerror() and report messages in whatever
+               # fashion is appropriate for the code in question.
                'perror',
                # Use PROTO_ITEM_SET_HIDDEN instead of these:
                'proto_tree_add_item_hidden',
@@ -161,6 +160,11 @@ my %APIs = (
                'g_error',
                ]},
 
+       # APIs that print to the terminal. Dissectors shouldn't call these
+       'termoutput' => { 'count_errors' => 0, 'functions' => [
+               'printf',
+               ]},
+
 );
 
 
index 8cd54cfa8588ed1f151c3374f63968c739eee27e..5bbb3595c9d5fb35b7a80102e85447952c27da36 100644 (file)
@@ -74,4 +74,4 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../tools/checkAPIs.pl -g abort $(NONGENERATED_C_FILES) $(GENERATOR_FILES)
+       $(PERL) ../tools/checkAPIs.pl -g abort -g termoutput $(NONGENERATED_C_FILES) $(GENERATOR_FILES)
index 3a304c5244b8a1e08dd769936802857feacb310f..743f86ad29e5709171c7b7198bbd1c12c27003d6 100644 (file)
@@ -53,6 +53,6 @@ distclean: clean
 maintainer-clean: distclean
 
 checkapi:
-       $(PERL) ../tools/checkAPIs.pl -g abort \
+       $(PERL) ../tools/checkAPIs.pl -g abort -g termoutput \
        $(LIBWSUTIL_SRC) \
 #      file_util.c