Re: [RFC/PATCH] Invokedynamic API stubs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 04, 2011 at 09:11:50AM +0100, Mark Wielaard wrote:
> Hi Pekka,
> 
> On Thu, 2011-02-03 at 16:47 +0200, Pekka Enberg wrote:
> > I'd like to check in these simple invokedynamic API stubs into CVS HEAD.
> > The APIs are not final but I think now is as good time as any to start
> > working on them especially as it needs work on the VM side. Furthermore,
> > there's already open source projects such as JRuby out there that use
> > invokedynamic so I think GNU Classpath probably needs to support it at
> > some point anyway.
> 
> > The classes don't do anything useful yet and don't even contain all
> > the specified methods.
> 
> I have to admit that I don't like putting in stubs. In the past we have
> decided to just leave things really unimplemented if we don't support
> it. And as you say it needs a bit more design thinking. So I would
> suggest you do this on a branch first (grmbl CVS...) and/or first try to
> spec it out against a jato VM implementation.
> 

I'd also rather see implementation than stubs :-)
Or if you do add stubs, please use the gnu.classpath.NotImplementedException
to mark them as such.

Also, as JDK7 stuff, I don't think this is 100% final yet.

There's lots of JDK6 stuff that needs doing :-D

> BTW. John Rose has lots of background material online:
> http://cr.openjdk.java.net/~jrose/pres/
> 
> Cheers,
> 
> Mark
> 
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37



[Index of Archives]     [Linux Kernel]     [Linux Cryptography]     [Fedora]     [Fedora Directory]     [Red Hat Development]

  Powered by Linux