[kaffe] CVS kaffe (doogie): Add in warnings produced by jikes.

Kaffe CVS Kaffe Mailing List <kaffe@kaffe.org>
Sat Apr 3 19:47:03 2004


PatchSet 4615 
Date: 2004/04/04 03:43:23
Author: doogie
Branch: HEAD
Tag: (none) 
Log:
Add in warnings produced by jikes.

Members: 
	ChangeLog:1.2192->1.2193 
	scripts/sort-warnings.pl:1.2->1.3 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.2192 kaffe/ChangeLog:1.2193
--- kaffe/ChangeLog:1.2192	Sun Apr  4 03:16:47 2004
+++ kaffe/ChangeLog	Sun Apr  4 03:43:23 2004
@@ -1,5 +1,9 @@
 2004-04-03  Adam Heath <doogie@debian.org>
 
+        * scripts/sort-warnings.pl: Add in warnings produced by jikes.
+
+2004-04-03  Adam Heath <doogie@debian.org>
+
 	* libraries/javalib/rebuildLib.in: Quote @JIKES@, so that
 	  additional arguments can be passed to it.
 
Index: kaffe/scripts/sort-warnings.pl
diff -u kaffe/scripts/sort-warnings.pl:1.2 kaffe/scripts/sort-warnings.pl:1.3
--- kaffe/scripts/sort-warnings.pl:1.2	Sun Apr  4 03:03:11 2004
+++ kaffe/scripts/sort-warnings.pl	Sun Apr  4 03:43:25 2004
@@ -6,12 +6,20 @@
 my $prefix_regex = qr/^([^:\n]+):(\d+): warning: /m;
 my $prefix_regex_noparam = qr/^(?:[^:\n]+):(?:\d+): warning: /m;
 my $prefix_regex2 = qr/^([^:\n]+):(\d+):(?:\d+): warning: /m;
+my $jikes_prefix = qr/([^:\n]+):(\d+):\d+:\d+:\d+: Semantic Warning: /m;
 
 #<robilad> guilhem: ~3000 unique ones with -Wall -W -Wtraditional -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wcast-qual
 #          -Wcast-align -Wwrite-strings -Wconversion -Wsign-compare -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes
 #          -Wmissing-declarations -Wmissing-noreturn -Wredundant-decls -Wnested-externs -Winline -Wlong-long
 
 my @warning_types = (
+	'java:throws-unchecked'		=> qr/${jikes_prefix}Since type "([^"]+)" is an unchecked exception, it does not need to be listed in the throws clause.$/m,
+	'java:modifier-order'		=> qr/${jikes_prefix}The modifier "([^"]+)" did not appear in the recommended order public\/protected\/private, abstract, static, final, synchronized, transient, volatile, strictfp.$/m,
+	'java:public-in-interface'	=> qr/${jikes_prefix}The use of the "([^"]+)" modifier in this context is redundant and strongly discouraged as a matter of style.$/m,
+	'java:exception-superclass'	=> qr/${jikes_prefix}The listing of type "([^"]+)" in the throws clause is not necessary, since its superclass, "([^"]+)", is also listed.$/m,
+	'java:override-default'		=> qr/${jikes_prefix}Method "([^"]+)" in class "([^"]+)" does not override or hide the corresponding method with default access in class "([^"]+)".$/m,
+	'java:invalid-zip'		=> qr/${jikes_prefix}The file "([^"]+)" does not exist or else is not a valid zip file.$/m,
+
 	'missing-prototypes-mismatch'	=> qr/${prefix_regex}no previous prototype for `([^']+)'\n${prefix_regex_noparam}type mismatch with previous implicit declaration\n${prefix_regex}previous implicit declaration of `[^']+'\n${prefix_regex_noparam}`[^']+' was previously implicitly declared to return `([^']+)'$/m,
 
 	'-Wformat-nonliteral'		=> qr/${prefix_regex}format not a string literal, argument types not checked$/m,