Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compilation of Enum on JDK 7 #1

Open
wants to merge 9 commits into
base: jb-mr1
Choose a base branch
from
Open

Fix compilation of Enum on JDK 7 #1

wants to merge 9 commits into from

Conversation

nocoast
Copy link

@nocoast nocoast commented Jan 12, 2013

Change-Id: I2e58b6a527e4ef1fc7ed3359b8aea88d01a031fb

The Android Automerger and others added 8 commits October 11, 2012 08:19
Change-Id: I1e5f377e655f72b8d2e497e83cf4207f5b786e01
Central directories of zip files should be read lazily.
They are needed only to service calls to
BaseDexClassLoader#findResouce(s). Most android processes
don't use these methods since they use android resources.

bug:6797061

Change-Id: I1a5b5d03572601707e1fb1fd4424c1ae2fd2217d
Change-Id: I2e58b6a527e4ef1fc7ed3359b8aea88d01a031fb
@Stevespear426
Copy link

Hey I'm not ignoring this I just haven't had time to make sure java 7 is all good I'll be getting to it soon. Also you can fix building with the newest Google tool chain by tracking my external_oprofile

@nocoast
Copy link
Author

nocoast commented Jan 14, 2013

Cool, I have more patches I would like to submit to help aid supporting jdk
7....at very least it compiles the source much faster, as long as you use
java6 code in your android builds you will be fine, the only real
vulnerability se 1.7 has is the fact that it wouldn't error if you wrote
code for 1.7....which is unsupported by android at this time...kinda minced
my words hope that makes sense...
On Jan 14, 2013 2:19 PM, "Steve Spear" [email protected] wrote:

Hey I'm not ignoring this I just haven't had time to make sure java 7 is
all good I'll be getting to it soon. Also you can fix building with the
newest Google tool chain by tracking my external_oprofile


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-12237600.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants