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

MOE Sync 2020-03-16 #443

Merged
merged 1 commit into from
Mar 16, 2020
Merged

MOE Sync 2020-03-16 #443

merged 1 commit into from
Mar 16, 2020

Conversation

cpovirk
Copy link
Member

@cpovirk cpovirk commented Mar 16, 2020

This code has been reviewed and submitted internally. Feel free to discuss on
the PR, and we can submit follow-up changes as necessary.

Commits:

Migrate from JSR-305 to the Checker Framework annotations

this is tangentially related to Java 11 preparedness.

dac42f8

this is tangentially related to Java 11 preparedness.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=300822964
@cpovirk cpovirk merged commit eb76f92 into master Mar 16, 2020
@cpovirk cpovirk deleted the sync-master-2020/03/16 branch March 16, 2020 14:59
@sormuras
Copy link
Contributor

this is tangentially related to Java 11 preparedness.

Including supporting new syntax elements (Java language features) as collected in #436?

@cpovirk
Copy link
Member Author

cpovirk commented Mar 16, 2020

Not sure what @cushon has in store, but CCing him in case he has anything to say.

@sormuras
Copy link
Contributor

Missed him at FOSDEM and the OCW in Brussels in early February.

@cushon
Copy link
Collaborator

cushon commented Mar 16, 2020

The language level changes in 11 (var, private interface methods) don't affect the parser/AST and should already be supported, and the features discussed in #436 were added or previewed after 11.

Supporting Java > 11 is still future work. In particular, there isn't an obvious way to do it without burning compatibility with Java <= 11 (which we aren't ready to do yet) or maintaining branches for each JDK release (which could happen but isn't free).

@sormuras
Copy link
Contributor

[...] maintaining branches for each JDK release (which could happen but isn't free).

Copied from https://twitter.com/BrianGoetz/status/1221884070155735040

I think you've still missed the point, which is: what you're wishing for, while reasonable, is far more costly than you're imagining.
Absent a serious effort to find a sponsor willing to underwrite this expense (for many years), is there more here than "I want a pony"?

I still want that pony -- and am interested in helping out on that case.

@cushon cushon mentioned this pull request Mar 16, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants