Skip to content

Refactor RenderManager for improved Type Safety, javadoc, and code clarity #2530

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

capdevon
Copy link
Contributor

@capdevon capdevon commented Jun 27, 2025

This PR modernizes the RenderManager class by improving type safety with generics, enhancing code readability, and updating inline documentation for clarity and completeness. Key changes include:

  • Replacing raw types with parameterized generics (e.g., RenderPipeline<?>, HashMap<Class<?>, PipelineContext>).
  • Simplifying and modernizing for-loops to use enhanced for-each syntax.
  • Refactoring viewport calculation variables to local scope, reducing class state.
  • Updating and expanding JavaDoc comments for public methods and parameters.
  • Improving code style and consistency (e.g., formatting, nullability, and assertion usage).
  • Minor bugfixes and code cleanup for better maintainability.

These changes collectively improve code quality, maintainability, and developer experience without altering core functionality.

@capdevon capdevon changed the title Update RenderManager javadoc Refactor RenderManager for improved Type Safety, javadoc, and code clarity Jun 27, 2025
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.

1 participant