Skip to content

Conversation

kwvanderlinde
Copy link
Collaborator

@kwvanderlinde kwvanderlinde commented Oct 9, 2025

Identify the Bug or Feature request

Completes #5780

Description of the Change

With the final change to 1.18, users can no longer enable the LibGDX renderer. Internally, the renderer is not actually used for anything either. Finally, development of the LibGDX renderer has moved to the feature-gdx-renderer branch for the time being. So that devs don't have to worry about updating LibGDX as they make other changes, this PR removes all the remaining GDX code from the codebase.

It leaves in a few bits of GDX support, notably the ZoneRenderer still checks of the GDX renderer is active before try to draw itself. This check is always false, but remains in the hopes of adding the GdxRenderer back in the future.

Possible Drawbacks

None

Documentation Notes

N/A

Release Notes

N/A


This change is Reviewable

The user can no longer enable it anyways, and development is continue on the `feature-gdx-renderer` branch.
@kwvanderlinde kwvanderlinde self-assigned this Oct 9, 2025
@kwvanderlinde kwvanderlinde linked an issue Oct 9, 2025 that may be closed by this pull request
@kwvanderlinde kwvanderlinde moved this from Todo to Awaiting-Review in MapTool 1.19 Oct 9, 2025
@kwvanderlinde kwvanderlinde added the code-maintenance Adding/editing javadocs, unit tests, formatting. label Oct 9, 2025
@github-project-automation github-project-automation bot moved this from Awaiting-Review to To-Be-Merged in MapTool 1.19 Oct 18, 2025
@kwvanderlinde kwvanderlinde added this pull request to the merge queue Oct 18, 2025
Merged via the queue into RPTools:develop with commit 318582b Oct 18, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from To-Be-Merged to Merged in MapTool 1.19 Oct 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-maintenance Adding/editing javadocs, unit tests, formatting.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

[Bug]: 1.18 performance issues

2 participants