osmocom
/
wireshark
11
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

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
This commit is contained in:
Gerald Combs 2008-06-27 22:48:25 +00:00
parent f8e3822738
commit b642ef10fa
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tools/checkAPIs.pl
View File

@ -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;
}
}

2
wsutil/Makefile.nmake
View File

@ -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