detail.php
OW2 Forge: Detail: 316375 Add version 52.0 in ASM to support JDK 8 classfiles

Advanced - Powered by Google


   
Log In
New Account
  
 
Home
My Page
Project Tree
Project Openings
ASM
          
 
 
Summary
Tracker
Lists
News
CVS
Files
SVN
              
 

Tracker: Feature Requests

Submit New | Browse | Admin | ExportToXml

[ #316375 ] Add version 52.0 in ASM to support JDK 8 classfiles

Date:
2012-11-23 05:58
Priority:
5
Submitted By:
Arul Dhesiaseelan (aruld)
Assigned To:
Nobody (None)
Category:
core
State:
Closed
Summary:
Add version 52.0 in ASM to support JDK 8 classfiles

Detailed description:
Accept classfiles with major version 52 for JDK 8 http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2012-October/004536.html Is it possible to add this new version support? IntelliJ IDEA is dependent on ASM 4, but since the new class file version was merged into lambda repository, it makes it difficult to use newer lambda builds with Intellij due to this bug (http://youtrack.jetbrains.com/issue/IDEA-95473). Attaching a patch that adds support for version 52 in ASM. I haven't tested it, hope someone can look into this.

Add A Comment:

Please login

Followup

Message
Date: 2012-12-01 19:22
Sender: thetaphi
Logged In: YES 
user_id=33164

At Apache Lucene, we use ASM 4 in our build system (not in 
Lucene itsself) to check the bytecode of our classes for 
invocations of discouraged methods (e.g. we don't allow 
calling methods like String#toLowerCase() without a Locale - 
see http://blog.thetaphi.de/2012/07/default-locales-default-
charsets-and.html). To do this check we check the class 
files emitted by javac with ASM by our custom ANT task. This 
works very good, and with the compiled class files, which 
are Java 6 compliant, we have no problems. Unfortunately, to 
do the full checks (needs digging into super classes), we 
load referred class files from classpath / rt.jar. Because 
of this, our build fails with Java 8 build 65.

It would be nice, if ASM 4.0 or 4.1 (in a bugfix release) 
could at least support the file format - without supporting 
the features, so simple loading of classes works. We don't 
need any new 52.0 features, just ClassReader should not fail 
with IAE (by the way, a IAE without a message is not good, 
you should explain why loading the class failed).

See the Lucene Core issue: 
https://issues.apache.org/jira/browse/LUCENE-4582
Date: 2012-11-25 23:13
Sender: aruld
Logged In: YES 
user_id=33120

Ah, my bad, I did not check the branches. Thanks for the update.
Date: 2012-11-24 17:32
Sender: ebruneton
Logged In: YES 
user_id=128

The prototype ASM version for JDK 8 is in the ASM_5_FUTURE
branch in the SVN repository.

Attached Files:

Name Description Download
jdk8.patch patch to support class version 52 to support JDK 8 classfiles Download

Changes:

Resource id #66listtable
status_id resolution_id close_date File Added
Field Old Value Date By
status_idOpen2013-10-12 08:39ebruneton
resolution_idNone2013-10-12 08:39ebruneton
close_date2013-10-12 08:392013-10-12 08:39ebruneton
File Added2339: jdk8.patch2012-11-23 06:06aruld

Copyright © 1999-2008, OW2 Consortium | contact | You've got a problem ? Please report an issue using your OW2 forge account credentials