On 9/24/07, Gregory Shimansky <[EMAIL PROTECTED]> 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++.
