Skip to content
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

JACOBIN-397 New javaUtilLocale.go; fix string trace #95

Merged
merged 4 commits into from
Nov 10, 2023
Merged

JACOBIN-397 New javaUtilLocale.go; fix string trace #95

merged 4 commits into from
Nov 10, 2023

Conversation

texadactyl
Copy link
Collaborator

@texadactyl texadactyl commented Nov 10, 2023

  • Added minimal support for Locale.
  • Fixed some boundary cases of string trace.
  • Implemented String.valueOf(...)

@texadactyl
Copy link
Collaborator Author

Note that the language comes from the LANGUAGE environment variable which is common to Linux/Unix, MacOS, and Windows.

@texadactyl texadactyl changed the title New javaUtilLocale.go; fix string trace JACOBIN-397 New javaUtilLocale.go; fix string trace Nov 10, 2023
@texadactyl
Copy link
Collaborator Author

Working on populating the String.valueOf(...) functions.

@texadactyl
Copy link
Collaborator Author

texadactyl commented Nov 10, 2023

Hey, 36 wins and only 60 losses!

One of the losses (stringer-valueof) got a consolation prize: evolved from panic to "INVOKEVIRTUAL: Native method requested: java/lang/Object.getClass". Not too shabby. Now, lets see if we can avoid the native function.

@platypusguy platypusguy merged commit 6f3a3b6 into platypusguy:main Nov 10, 2023
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