Skip to content

Conversation

@louhy
Copy link
Contributor

@louhy louhy commented Jun 6, 2019

Took way longer than I thought it would for "core" so I didn't get to "vr" yet, but OpenJDK 11 builds produce far fewer warnings after this cleanup. (OpenJDK 8 also tested to make sure new warnings don't appear...) I took the time to understand & preserve comment intent as best I could, and test JavaDoc rendering in NB 11 to ensure doc is readable (before fixes some comments were showing completely misleading text in autocomplete help!). Also fixed a few broken links.

Also here "ContextListener" was changed to SystemListener - I can't find any "ContextListener" other than in some comments (possibly also outdated), so I think this was a typo/mistake?
If not and there is some "ContextListener" concept I'm unaware of, let me know and I'll correct.

/**
 * The <code>ContextListener> provides a means for an application  (-)
 * The {@code SystemListener} provides a means for an application  (+)
 * to receive events relating to a context.
 */
public interface SystemListener {

@stephengold stephengold merged commit f8dae05 into jMonkeyEngine:master Jun 9, 2019
@stephengold stephengold added this to the v3.2.4 milestone Jul 8, 2019
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