Skip to content

JACOBIN-910, JACOBIN-911#481

Merged
platypusguy merged 3 commits into
platypusguy:mainfrom
texadactyl:main
May 3, 2026
Merged

JACOBIN-910, JACOBIN-911#481
platypusguy merged 3 commits into
platypusguy:mainfrom
texadactyl:main

Conversation

@texadactyl
Copy link
Copy Markdown
Collaborator

@texadactyl texadactyl commented May 2, 2026

JACOBIN-910
object/javaByteArray.go: Junie performance enhancement of 2 functions
gfunction/javaLang/javaLangString.go: Junie protection of byte handling
gfunction/javaLang/javaLangStringBuilder.go: Junie protection of byte handling
jvm/runUtils.go: addition of function decodeModifiedUTF8
jvm/interpreter.go: amend function ldc() to use decodeModifiedUTF8 in processing Java Strings

JACOBIN-911
gfunction/javaLang/javaLangStringBuilder.go - fix charAt
gfunction/javaLang/javaLangStringBuilder_test.go - fix unit tests

@texadactyl texadactyl requested a review from platypusguy May 2, 2026 19:43
@texadactyl
Copy link
Copy Markdown
Collaborator Author

texadactyl commented May 2, 2026

5 O/Ses
Number of discrepancies: 0
Passed-Failed = 280-23 (92.4%)

with the addition of the 2 leibniz test cases (both pass) and the 2 mindrot cases (both fail)

@texadactyl
Copy link
Copy Markdown
Collaborator Author

JACOBIN-910 is fixed.
My "nice try" remark is about JACOBIN-911.

@texadactyl
Copy link
Copy Markdown
Collaborator Author

Got JACOBIN-911 too!

@texadactyl texadactyl changed the title JACOBIN-910 fix Jacobin handling of "\000" JACOBIN-910, JACOBIN-911 May 3, 2026
@texadactyl
Copy link
Copy Markdown
Collaborator Author

5 O/Ses
Number of discrepancies: 0
Passed-Failed = 282-21 (93.1%)

@platypusguy platypusguy merged commit 7d145db into platypusguy:main May 3, 2026
3 checks passed
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