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

Fix #2911: javalib IPv6 addresses now display using only lowercase hexadecimal digits #2913

Conversation

LeeTibbert
Copy link
Contributor

@LeeTibbert LeeTibbert commented Oct 18, 2022

Fix #2911

When javalib displays an IPv6 address it now matches the JVM output, e.g. "fe80::mumble".

The content of this PR is a candidate for porting to the 0.4.x stream. Because of the
restructuring in merged 0.5.0 PR #2877, backporting this PR should probably happen
after #2877 is backported.

@LeeTibbert LeeTibbert added component:javalib backport candidate PR which might be backported into previous major release of SN labels Oct 18, 2022
@LeeTibbert
Copy link
Contributor Author

This PR is ready for review & merge. Thank you for considering it.

The three build failures have nothing to do with the change of this PR.
I know, I say that every PR and am going to wear it out.

@LeeTibbert
Copy link
Contributor Author

This PR will make it easier for me & others to write unit-tests which match ScalaJVM.

@LeeTibbert
Copy link
Contributor Author

3 build failures reviewed. Network connection failures, not related to the substance of this PR.

I believe this PR is ready for review & merge. Thank you.

@WojciechMazur WojciechMazur merged commit c739841 into scala-native:main Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport candidate PR which might be backported into previous major release of SN component:javalib
Projects
None yet
Development

Successfully merging this pull request may close these issues.

javalib Inet6Address should format IPv6 numeric addresses using lower case letters.
2 participants