Skip to content

remove nodeType from Branch nodes (its hardly used) this reduces the retained memory of nodes (4 or 8 bytes per Mode) #1

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.

Sign up for GitHub

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

Jump to bottom
Closed
mkeskells wants to merge 1 commit into from
Closed

remove nodeType from Branch nodes (its hardly used) this reduces the retained memory of nodes (4 or 8 bytes per Mode) #1

mkeskells wants to merge 1 commit into from
+141 −144

Conversation

@mkeskells
Copy link
Owner

@mkeskells mkeskells commented Aug 10, 2025

SUMMARY

  • Describe your changes, including rationale and design decisions
  • If your pull request includes new functionality, then you must have corresponding new tests.
  • We require all Java files to be ASCII-only to avoid compatibility problems.

Automated Checks

  • I have run ./gradlew test and made sure that my PR does not break any unit test.

@mkeskells
remove nodeType from Branch nodes (its hardly used)
062ee88 
this reduces the retained memory of nodes (4 or 8 bytes per Mode)
@mkeskells mkeskells closed this Aug 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mkeskells