Skip to content

Generate async methods with synthetic access modifier#31

Merged
JoeHegarty merged 1 commit intoelectronicarts:masterfrom
elventear:synthetic
Dec 5, 2018
Merged

Generate async methods with synthetic access modifier#31
JoeHegarty merged 1 commit intoelectronicarts:masterfrom
elventear:synthetic

Conversation

@elventear
Copy link
Contributor

ea-async generated bytecode can create problems with tools that analyze
the java bytecode directly without looking at the source code. Usually
these tools ignore bytecode methods that are marked as synthetic since
that means it is compiler generated code not written directly by a user.

This change ensures that all methods generated are marked as synthetic,
which fixes issues with aforementioned tools.

ea-async generated bytecode can create problems with tools that analyze
the java bytecode directly without looking at the source code. Usually
these tools ignore bytecode methods that are marked as synthetic since
that means it is compiler generated code not written directly by a user.

This change ensures that all methods generated are marked as synthetic,
which fixes issues with aforementioned tools.
@JoeHegarty
Copy link
Member

Awesome, thanks. I should be able to get this in a release later this week.

@JoeHegarty JoeHegarty merged commit 9434cba into electronicarts:master Dec 5, 2018
@elventear elventear deleted the synthetic branch December 7, 2018 17:36
@elventear
Copy link
Contributor Author

@JoeHegarty would it be possible to do a release today?

@JoeHegarty
Copy link
Member

Release is cut.
https://github.com/electronicarts/ea-async/releases/tag/v1.2.2

Should be in Maven Central shortly.

@elventear
Copy link
Contributor Author

Awesome. Thanks!

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.

2 participants