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

Made static members final so it's compatible with GWT. #156

Merged
merged 1 commit into from
Apr 9, 2015

Conversation

dfumosa
Copy link
Contributor

@dfumosa dfumosa commented Apr 8, 2015

Was trying to compile HTML with GWT in LibGDX but it complained about these members not being final. So I propose a small change to make it compatible. Please let me know if there are any reasons why this isn't a good idea. Thanks!

@metaphore
Copy link
Contributor

Derp, that what I fixed once... but #148 broke it again. Perhaps we need to add some comments over that methods to prevent such situation in future?

@dsaltares
Copy link
Member

Sorry about that, shouldn't have let that PR go it as it was.

dsaltares added a commit that referenced this pull request Apr 9, 2015
Made static members final so it's compatible with GWT.
@dsaltares dsaltares merged commit 536a12a into libgdx:master Apr 9, 2015
@dsaltares dsaltares added the bug label Apr 9, 2015
@junkdog
Copy link
Contributor

junkdog commented Apr 9, 2015

Derp, that what I fixed once... but #148 broke it again. Perhaps we need to add some comments over that methods to prevent such situation in future?

Maybe setup a jenkins job to compile an ashley GWT project every now and then? Most GWT errors surface during the compilation phase.

@metaphore metaphore mentioned this pull request May 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants