harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pavel Pervov (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-2528) [drlvm] VM crashes instead of throwing AbstractMethodError
Date Fri, 08 Dec 2006 17:02:25 GMT
    [ http://issues.apache.org/jira/browse/HARMONY-2528?page=comments#action_12456911 ] 
            
Pavel Pervov commented on HARMONY-2528:
---------------------------------------

Alexei, as per your request, I reviewed your patch. First of all it contains lots of unrelated
changes. I would suggest filing separate JIRA against changes requested in http://wiki.apache.org/harmony/Code_Inspection%3a_compile%2ecpp.

Commenting your fix, I would suggest making these changes in vm/vmcore/src/class_support/Prepare.cpp.
There should be 'if' inserted for abstract methods where compile_me_stubs are assigned into
methods' code addresses. Respective stub for throwing AME is already available there. Actually,
I'll attach alternative patch by myself.

> [drlvm] VM crashes instead of throwing AbstractMethodError
> ----------------------------------------------------------
>
>                 Key: HARMONY-2528
>                 URL: http://issues.apache.org/jira/browse/HARMONY-2528
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>            Reporter: Alexei Fedotov
>         Attachments: AbstractMethodError.patch, AbstractMethodError_1.patch, AbstractMethodError_2.patch,
Test1.java
>
>
> The following test perfectly compiles with javac, and throws 
> java.lang.AbstractMethodError: javax/management/monitor/Monitor.createMX4JMonitor()Lmx4j/monitor/MX4JMonitor;
>         at javax.management.monitor.Monitor.getMX4JMonitor(Monitor.java:54)
>         at javax.management.monitor.Monitor.getObservedObjects(Monitor.java:149)
>         at Test1.main(Test1.java:7)
> when used with mx4j implementation. DRLVM + JIT crashes trying to compile abstract method.
> import javax.management.monitor.*;
> import javax.management.*;
> public class Test1 {
>    public static void main(String[] args) throws Exception {
>        (new MonitorImpl()).getObservedObjects();
>    }
> }
> class MonitorImpl extends javax.management.monitor.Monitor {
>     @Override
>     public void start() {}
>     @Override
>     public void stop() {}
> }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message