[kaffe] hsqldb - Abstract method in non-abstract class error

[Rob Gonzalez]

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
> kaffe at kaffe.org
> http://kaffe.org/cgi-bin/mailman/listinfo/kaffe
>