[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,