Skip to content

Fixes the excessive memory usage described in GH-47 - #48

Open
gioyous-it wants to merge 6 commits into
Bixilon:masterfrom
gioyous-it:master
Open

Fixes the excessive memory usage described in GH-47#48
gioyous-it wants to merge 6 commits into
Bixilon:masterfrom
gioyous-it:master

Conversation

@gioyous-it

Copy link
Copy Markdown

Fixes the excessive memory usage described in GH-47.

(Fixing this PR.)

@gioyous-it

Copy link
Copy Markdown
Author

Small intendation bug. Fixing.

@gioyous-it

Copy link
Copy Markdown
Author

it's starting to piss me off, no idea what broke

@gioyous-it

Copy link
Copy Markdown
Author

I hate the GitHub website. No idea how that duplicated.

@gioyous-it

Copy link
Copy Markdown
Author

Maybe patched?? But It broke another file in the process

@gioyous-it

Copy link
Copy Markdown
Author

Most likely patched.

@Bixilon

Bixilon commented Aug 29, 2026

Copy link
Copy Markdown
Owner

You can amend commits (I am developing locally with git).

Why should that fix the excessve allocation? We just skip chars that are already mapped (but in a bad order, it should be last in, not first in). Why are chars even duplicated? Are you using a resource pack?

@gioyous-it

Copy link
Copy Markdown
Author

You can amend commits (I am developing locally with git).

Why should that fix the excessve allocation? We just skip chars that are already mapped (but in a bad order, it should be last in, not first in). Why are chars even duplicated? Are you using a resource pack?

I don't even know myself. Lol.
I've found out this Is the exact Reason It crashes

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