Skip to content

Simple Cleanups#61

Merged
Dream-Master merged 9 commits into
masterfrom
simple-cleanups
Jun 19, 2026
Merged

Simple Cleanups#61
Dream-Master merged 9 commits into
masterfrom
simple-cleanups

Conversation

@Nikolay-Sitnikov

Copy link
Copy Markdown
Contributor

Cleans up the code a tiny bit, moving comments around, removing some @SuppressWarnings and some unused code, and using modern Java features.

These changes should require no modifications to any outside mods.

@Nikolay-Sitnikov Nikolay-Sitnikov added the Enhancement Improve an existing mechanic. Please explain the change with a before/after comparison. label May 27, 2026

@Glease Glease left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

relicense to GPL without notice is not going to work

@Glease Glease left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually, never mind. I see what you mean. TIL

@Glease Glease self-requested a review May 28, 2026 01:32
@Glease

Glease commented May 28, 2026

Copy link
Copy Markdown
Contributor

ok I remember why we were stuck with primitive java syntax. the javadoc generator is from java 11 so it doesn't really support the nice syntax sugars from newer java. I like the frames so I kinda want to stay with it...

@Nikolay-Sitnikov

Copy link
Copy Markdown
Contributor Author

Surely we can find a modern Javadoc generator to use in our buildscript?

CC @eigenraven - Do you know if this is easily possible?

@Nikolay-Sitnikov

Copy link
Copy Markdown
Contributor Author

ok I remember why we were stuck with primitive java syntax. the javadoc generator is from java 11 so it doesn't really support the nice syntax sugars from newer java. I like the frames so I kinda want to stay with it...

Well, https://github.com/climber09/Javadoc-Frames-Generator/tree/master exists. Unfortunately, it'd be a massive pain to run in our buildscript as-is.

@boubou19

Copy link
Copy Markdown
Member

If we need java 25 directly in the dev env, we can always enable jvmdowngrader in the buildscript and get away with it.

@Nikolay-Sitnikov

Copy link
Copy Markdown
Contributor Author

If we need java 25 directly in the dev env, we can always enable jvmdowngrader in the buildscript and get away with it.

Java 25 doesn't have framed Javadocs either (although they're apparently much better than the old non-framed Javadocs), and we can enable Java 25 for the Javadoc task individually in the buildscript anyways.

@Nikolay-Sitnikov

Copy link
Copy Markdown
Contributor Author

@Glease Would you be alright with modern Java 25 javadocs?

@Glease

Glease commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

In the end i'll not be the one who read the doc, so a dev vote feels appropriate. please build an example of java 25 doc and get the pipeline to build it, then we can do a proper vote

@Dream-Master Dream-Master requested a review from a team as a code owner June 17, 2026 09:22
@Dream-Master Dream-Master merged commit 2c5fa02 into master Jun 19, 2026
1 check passed
@Dream-Master Dream-Master deleted the simple-cleanups branch June 19, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Improve an existing mechanic. Please explain the change with a before/after comparison.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants