diff options
| author | Gerald Combs <gerald@wireshark.org> | 2008-06-27 22:48:25 +0000 |
|---|---|---|
| committer | Gerald Combs <gerald@wireshark.org> | 2008-06-27 22:48:25 +0000 |
| commit | b642ef10fa8902ea4b9978a7deb82c8041cd7b43 (patch) | |
| tree | e847a4694771f14f9f367aea1bdb5f8925dfd72b | |
| parent | f8e3822738c2c6c906c1927ebe88c639c766a725 (diff) | |
| download | wireshark-b642ef10fa8902ea4b9978a7deb82c8041cd7b43.tar.gz wireshark-b642ef10fa8902ea4b9978a7deb82c8041cd7b43.tar.bz2 wireshark-b642ef10fa8902ea4b9978a7deb82c8041cd7b43.zip | |
Have checkAPIs.pl print to stderr, so that its output is colored red in
Buildbot. Make capitalization more consistent. Look for abort functions in
wsutil.
svn path=/trunk/; revision=25619
| -rwxr-xr-x | tools/checkAPIs.pl | 10 | ||||
| -rw-r--r-- | wsutil/Makefile.nmake | 2 |
2 files changed, 6 insertions, 6 deletions
diff --git a/tools/checkAPIs.pl b/tools/checkAPIs.pl index 4a54d018e0..d6d90d1713 100755 --- a/tools/checkAPIs.pl +++ b/tools/checkAPIs.pl @@ -233,7 +233,7 @@ while ($_ = $ARGV[0]) if ($fileContents =~ m{[\x80-\xFF]}) { - print "Warning: found non-ASCII characters in " .$filename."\n"; + print STDERR "Warning: Found non-ASCII characters in " .$filename."\n"; # Treat as warning # $errorCount++; } @@ -241,13 +241,13 @@ while ($_ = $ARGV[0]) if ($fileContents =~ m{%ll}) { # use G_GINT64_MODIFIER instead of ll - print "Error: found %ll in " .$filename."\n"; + print STDERR "Error: Found %ll in " .$filename."\n"; $errorCount++; } if (! ($fileContents =~ m{\$Id.*\$})) { - print "Warning: ".$filename." does not have an SVN Id tag.\n"; + print STDERR "Warning: ".$filename." does not have an SVN Id tag.\n"; } # Remove all the C-comments and strings @@ -255,7 +255,7 @@ while ($_ = $ARGV[0]) if ($fileContents =~ m{//}) { - print "Error: Found C++ style comments in " .$filename."\n"; + print STDERR "Error: Found C++ style comments in " .$filename."\n"; $errorCount++; } @@ -271,7 +271,7 @@ while ($_ = $ARGV[0]) $pfx = "Error"; } - print $pfx . ": Found " . $apiName . " APIs in ".$filename.": ".join(',', @foundAPIs)."\n" if @foundAPIs; + print STDERR $pfx . ": Found " . $apiName . " APIs in ".$filename.": ".join(',', @foundAPIs)."\n" if @foundAPIs; } } diff --git a/wsutil/Makefile.nmake b/wsutil/Makefile.nmake index 229a6a7709..3a304c5244 100644 --- a/wsutil/Makefile.nmake +++ b/wsutil/Makefile.nmake @@ -53,6 +53,6 @@ distclean: clean maintainer-clean: distclean checkapi: - $(PERL) ../tools/checkAPIs.pl \ + $(PERL) ../tools/checkAPIs.pl -g abort \ $(LIBWSUTIL_SRC) \ # file_util.c |