harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rana Dasgupta" <rdasg...@gmail.com>
Subject Re: [general] reminder: M3 feature freeze
Date Mon, 24 Sep 2007 19:51:02 GMT
On 9/24/07, Gregory Shimansky <gshimansky@gmail.com> wrote:
> Alexei Zakharov wrote:
> > Hi all,
> >
> >>> 2) org.apache.harmony.beans.tests.java.beans.beancontext.BeanContextSupportTest
> >> This passes ok for me.
> >
> > I've just fixed this failure. Looks like it was introduced by commit
> > r578410 - most likely the patch haven't been checked on DRLVM. The
> > problem itself was in differences between JNI implementations in DRLVM
> > and J9/RI. For more details please see HARMONY-4840 and HARMONY-4841.
>
> Alexei, thank you for finding this bug.
>
> I created a patch for HARMONY-4841 that fixes CallVoidMethod
> implementation in DRLVM when calling constructors. Would anyone mind if
> I commit it?
>
  This is a good fix. Invoking constructors virtually is not correct,
though constructors can make virtual calls. Unlike C++.

Mime
View raw message