Re: Mauve patches for SpinnerListModel

Re: Mauve patches for SpinnerListModel

[Mark Wielaard]

[-- Attachment #1: Type: text/plain, Size: 2255 bytes --]

Hi,

On Thu, 2004-08-19 at 02:37, Andrew John Hughes wrote:
> Attached is a series of Mauve tests for SpinnerListModel
> to go in the directory gnu/testlet/javax/swing/SpinnerListModel.
> What do I do about adding these?

You post them to mauve-patches@sources.redhat.com :)
And/Or you ask Tom for permission to check them in yourself.

> Also, does anyone have any idea as to why Mauve only seems to test java.io and
> java.net when I run make check?  Using ant, I can get it to compile all the tests
> but it segfaults in java.lang.reflect.Array.newInstance.

For these kind of questions there is the
mauve-discuss@sources.redhat.com mailinglist.
Personally I don't use ant but the batch_run and runner script (see the
README) which should automatically run all the tests. The README also
has some more examples on how to run mauve "the traditional way" (make
and autotools).

The crash in Array.newInstance() is a bug in jamvm (which I assume you
are using). CCed Robert. I thought I had reported it earlier, but maybe
I forgot. This is in the last multi-dimensional test:

    val = 0;
    try
      {
        x = Array.newInstance(String.class, new int[]
          {Integer.MAX_VALUE, Integer.MAX_VALUE});
        val = 1;
      }
    catch (OutOfMemoryError e)
      {
        val = 2;
      }
    catch (Throwable t)
      {
        harness.debug(t);
        val = 3;
      }
    harness.check(val, 2);

gdb backtrace below.

This is why I run the batch_run script since it will just ignore such
crashes and go on with the next test.

Cheers,

Mark

isInstOfArray0 (array_class=0x401bd440, test_elem=0x0, test_dim=2) at cast.c:52
52              return IS_INTERFACE(CLASS_CB(array_elem)) ?
(gdb) bt
#0  isInstOfArray0 (array_class=0x401bd440, test_elem=0x0, test_dim=2)
    at cast.c:52
#1  0x0804ac5b in arrayStoreCheck (array_class=0x0, test=0x0) at cast.c:85
#2  0x0805227e in executeJava () at interp.c:820
#3  0x0804e532 in executeMethodVaList (ob=0x0, class=0x0, mb=0x8119260,
    jargs=0xbfffdd40 "\034\uffff\032@\001") at execute.c:66
#4  0x0804e3c6 in executeMethodArgs (ob=0x0, class=0x0, mb=0x0) at execute.c:38
#5  0x080565d2 in main (argc=2, argv=0xbfffdde4) at jam.c:278

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: Mauve patches for SpinnerListModel

[Robert Lougher]

Hi,

I was all ready to release JamVM 1.2.0 but I saw this :)  I've fixed
it, but it's too late to get a release out tonight now (it's 3:30am
and it's work tomorrow).  I'll hopefully get my brother to put it up
on Sourceforge for me tomorrow.

Rob.

On Sun, 05 Sep 2004 21:26:08 +0200, Mark Wielaard <mark@klomp.org> wrote:
> Hi,
> 
> On Thu, 2004-08-19 at 02:37, Andrew John Hughes wrote:
> > Attached is a series of Mauve tests for SpinnerListModel
> > to go in the directory gnu/testlet/javax/swing/SpinnerListModel.
> > What do I do about adding these?
> 
> You post them to mauve-patches@sources.redhat.com :)
> And/Or you ask Tom for permission to check them in yourself.
> 
> > Also, does anyone have any idea as to why Mauve only seems to test java.io and
> > java.net when I run make check?  Using ant, I can get it to compile all the tests
> > but it segfaults in java.lang.reflect.Array.newInstance.
> 
> For these kind of questions there is the
> mauve-discuss@sources.redhat.com mailinglist.
> Personally I don't use ant but the batch_run and runner script (see the
> README) which should automatically run all the tests. The README also
> has some more examples on how to run mauve "the traditional way" (make
> and autotools).
> 
> The crash in Array.newInstance() is a bug in jamvm (which I assume you
> are using). CCed Robert. I thought I had reported it earlier, but maybe
> I forgot. This is in the last multi-dimensional test:
> 
>     val = 0;
>     try
>       {
>         x = Array.newInstance(String.class, new int[]
>           {Integer.MAX_VALUE, Integer.MAX_VALUE});
>         val = 1;
>       }
>     catch (OutOfMemoryError e)
>       {
>         val = 2;
>       }
>     catch (Throwable t)
>       {
>         harness.debug(t);
>         val = 3;
>       }
>     harness.check(val, 2);
> 
> gdb backtrace below.
> 
> This is why I run the batch_run script since it will just ignore such
> crashes and go on with the next test.
> 
> Cheers,
> 
> Mark
> 
> isInstOfArray0 (array_class=0x401bd440, test_elem=0x0, test_dim=2) at cast.c:52
> 52              return IS_INTERFACE(CLASS_CB(array_elem)) ?
> (gdb) bt
> #0  isInstOfArray0 (array_class=0x401bd440, test_elem=0x0, test_dim=2)
>     at cast.c:52
> #1  0x0804ac5b in arrayStoreCheck (array_class=0x0, test=0x0) at cast.c:85
> #2  0x0805227e in executeJava () at interp.c:820
> #3  0x0804e532 in executeMethodVaList (ob=0x0, class=0x0, mb=0x8119260,
>     jargs=0xbfffdd40 "\034\uffff\032@\001") at execute.c:66
> #4  0x0804e3c6 in executeMethodArgs (ob=0x0, class=0x0, mb=0x0) at execute.c:38
> #5  0x080565d2 in main (argc=2, argv=0xbfffdde4) at jam.c:278
> 
> 
> 
>

Re: Mauve patches for SpinnerListModel

[Thomas Zander]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Sunday 05 September 2004 21:26, Mark Wielaard wrote:
> > Also, does anyone have any idea as to why Mauve only seems to test
> > java.io and java.net when I run make check?  Using ant, I can get it
> > to compile all the tests but it segfaults in
> > java.lang.reflect.Array.newInstance.
>
> For these kind of questions there is the
> mauve-discuss@sources.redhat.com mailinglist.

Immediately after reporting the bug :) you can alter the build.xml to only 
test the classes you wrote quite easilly.
In the "test" target there are some <fileset /> lines and all you have to 
do is to make sure the includes part points to the dir you want to test 
(it will take all subclasses as well if you use 2 *'s at the end)

Naturally you should remove the fileset tags you don't want (there are 2 by 
default)

- -- 
Regards
Thomas
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFBPI64CojCW6H2z/QRAmaQAKCEUrxJ5Fj+wL8V8PFxt8DWnrifqQCff/0o
HZHR71RZTMXYc6BzUWtPof8=
=aAyF
-----END PGP SIGNATURE-----