Skip to content

KeyAction: use static KeyNames.getName() instead of instantiating class.#100

Open
Ali-RS wants to merge 1 commit intojMonkeyEngine-Contributions:masterfrom
Ali-RS:fix-illegal-access-error
Open

KeyAction: use static KeyNames.getName() instead of instantiating class.#100
Ali-RS wants to merge 1 commit intojMonkeyEngine-Contributions:masterfrom
Ali-RS:fix-illegal-access-error

Conversation

@Ali-RS
Copy link
Copy Markdown
Contributor

@Ali-RS Ali-RS commented Apr 5, 2021

No description provided.

@stephengold
Copy link
Copy Markdown

I'm reverting the change to jme3-core that caused the breakage. This PR is still a good idea, but not so urgent any more.

@Ali-RS
Copy link
Copy Markdown
Contributor Author

Ali-RS commented Nov 25, 2022

@pspeed42 still interested in this PR?

@pspeed42
Copy link
Copy Markdown
Contributor

I really wish I had a better workflow to notice these messages from GIT... this one was nicely directly in my main in-box at least.

The new way is definitely "better" in some ways. I don't have a strong opinion beyond that.

@pspeed42
Copy link
Copy Markdown
Contributor

I'm finally trying to catch up to all of my open pull requests but this one seems to be in a weird state. Sorry I've been so delinquent in applying these patches. :(

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.

3 participants