Hi, The check for abstract methods in non-abstract classes is actually something that _should_ take place in pass 2 of verification. Currently, it's checked in classMethod.c:682.
Sun's JVM does not enforce this rule at all, so unless someone has an objection I say we go ahead and remove that check entirely. I can't see any technical problems with this change and have regression tested the latest CVS head with KJC and it passes everything. Rob On Thu, 31 Jul 2003, Jim Pick wrote: > Hi, > > I'd like to use hsqldb 1.7.1 for the simple regression testing reporting > thing I'm writing to install on kaffe.org, but I'm running into a bug. > > It's super cool, BTW. Get it here: > > http://hsqldb.sourceforge.net/ > > Here's a trivial example, that creates a SQL table in-memory. > > import java.sql.*; > public class TestHSQLDB { > public static void main(String[] args) > throws ClassNotFoundException, SQLException { > Class.forName("org.hsqldb.jdbcDriver"); > Connection con = DriverManager.getConnection( > "jdbc:hsqldb:.", "sa", ""); > Statement stmt = con.createStatement(); > stmt.executeUpdate("create table TestResults " + > "(codebase varchar(50), " + > " branch varchar(50), " + > " change integer, " + > " category varchar(128), " + > " status varchar(20) )"); > } > } > > $ kaffe -cp .:../webapp/WEB-INF/lib/hsqldb.jar TestHSQLDB > java.lang.ClassFormatError: (class: org/hsqldb/jdbcConnection, method: > setHoldability signature: (I)V) Abstract method in non-abstract class > at java.sql.DriverManager.getConnection (DriverManager.java:164) > at java.sql.DriverManager.getConnection (DriverManager.java:188) > at TestHSQLDB.main (TestHSQLDB.java:10) > > This blocks me, so I'll definitely dive into it this weekend - but I > thought somebody on the list might be able to fix it faster. > > Cheers, > > - Jim > > _______________________________________________ > kaffe mailing list > [EMAIL PROTECTED] > http://kaffe.org/cgi-bin/mailman/listinfo/kaffe > _______________________________________________ kaffe mailing list [EMAIL PROTECTED] http://kaffe.org/cgi-bin/mailman/listinfo/kaffe
