Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: pass (asf.osuosl.org: domain of ivan.volosyuk@gmail.com
 designates 64.233.166.181 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=DKFBxz6GnK8o9oEy+C+j5fC4YE3jKHV/HpS3tyrgVkoh+IUGq2CIxqxzF5w8MhRgIq3VGmtyaecNWiy5/QZRCLsddekOMom8FYYeL7XevB6LZYc0GX1lmIyLFjv06/ILt2TZNyzCh6ydM+CYba4bY9XmGLgsJKqqbA9Ciox0eVQ=
Message-ID: <12385bbd0606060405k9135623x8e7f621a6f0b1771@mail.gmail.com>
Date: Tue, 6 Jun 2006 15:05:10 +0400
From: "Ivan Volosyuk" <ivan.volosyuk@gmail.com>
To: harmony-dev@incubator.apache.org
Subject: Re: [general] revisiting structure of SVN
In-Reply-To: <448555F1.6010309@googlemail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <906dd82e0606051940id5557a1ha050d6a72fa2ddc9@mail.gmail.com>
	 <200606060555.k565tBdZ021983@d06av02.portsmouth.uk.ibm.com>
	 <906dd82e0606052320n2db6552ayc0587a6cf2ea6e17@mail.gmail.com>
	 <448544FB.6050404@googlemail.com>
	 <906dd82e0606060300k2b410e9ex62cc9c0a52855f06@mail.gmail.com>
	 <448555F1.6010309@googlemail.com>

2006/6/6, Oliver Deakin <oliver.deakin@googlemail.com>:
> Mikhail Loenko wrote:
> > 2006/6/6, Oliver Deakin <oliver.deakin@googlemail.com>:
> >> Mikhail Loenko wrote:
> >> > They are not just stubs, I remember I've fixed a bug there [1].
> >>
> >> Even though you fixed the code there, it wont be getting used when you
> >> run with
> >> the classlib launcher. The luni-kernel and security-kernel classes under
> >> modules are
> >> only used to compile against, and the luni-kernel-stubs.jar and
> >> security-kernel-stubs.jar
> >> that they are packaged up into are never put on the bootclasspath (see
> >> depends/files/bootclasspath.properties).
> >
> > It's funny because my fix had solved the problem :)
>
> I was thinking about that, so I had a look back at the mail archives and
> the JIRA you linked. It looks like a new version of the VME came out
> around the time that you were working on the bug. I think there might
> have been a massive coincidence that you got the new VME, which
> contained a similar fix in its kernel classes, and used this to make
> a final run of the test against Harmony code, which then passed.
> Quite impressive timing! ;)
>
> >
> > If they are just stubs why not remove code from them? And why not to
> > put them
> > to depends or whatever to not confuse people?
>
> I think it's worth keeping the code there, just as a starting point for VM
> writers to implement their own kernel classes. It should be possible for
> a VM writer to adapt them for their own kernel, thus lessening the initial
> amount of time it takes to get the VM working with Harmony classlib.
>
> I can see how confusion could be caused by their location, however
> IMHO it would cause more confusion to have the kernel stubs
> located separate to the rest of the Java code.
> Perhaps it would be clearer if the directories were renamed
> luni-kernel-stubs and security-kernel-stubs (which would also match
> the jar names they generate)?
>
> Regards,
> Oliver

A small note.
As a writter of classlib adapter for jchevm I can say that the stubs
was quite useful for writting kernel classes specific for jchevm.
There are also some troubles with them: number of functions fallbacks
to null value, while it can be something like RuntimeException("not
implemented"). Some of the classes has better implementation in drlvm,
which can also be used as default implementation.

-- 
Ivan
Intel Middleware Products Division

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org