Skip to content

Issue #1909 - remove deprecated Unsafe usage#1932

Open
kamilkrzywanski wants to merge 1 commit intogoogle:masterfrom
kamilkrzywanski:issue_1909
Open

Issue #1909 - remove deprecated Unsafe usage#1932
kamilkrzywanski wants to merge 1 commit intogoogle:masterfrom
kamilkrzywanski:issue_1909

Conversation

@kamilkrzywanski
Copy link
Copy Markdown

@kamilkrzywanski kamilkrzywanski commented Apr 18, 2026

(requires jdk15+)
it will also resolve issue #1901

@google-cla
Copy link
Copy Markdown

google-cla Bot commented Apr 18, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

(requires at least jdk15)

Signed-off-by: Kamil Krzywanski <kamilkrzywanski01@gmail.com>
@GedMarc
Copy link
Copy Markdown

GedMarc commented Apr 19, 2026

@kamilkrzywanski
Copy link
Copy Markdown
Author

@GedMarc Where can I find parent pom to run this project?
UnsafeClassDefiner in my code is renamed to LookupClassDefiner, AnonymousClassDefiner is dead and removed since Unsafe::defineAnonymousClass is removed

@MarcMagonMNS
Copy link
Copy Markdown

Hmm snapshots don't go to central anymore - I'm pushing through RC6, central is a nightmare will ping when up.
I use the github packages

<repository>
    <id>guicedee-github</id>
    <url>https://maven.pkg.github.com/GuicedEE/</url>
    <snapshots>
        <enabled>true</enabled>
    </snapshots>
</repository>

Those are actually better resolutions than I had, I used the private lookups but it looks cleaner

@kamilkrzywanski
Copy link
Copy Markdown
Author

The solution based on https://github.com/GuicedEE/Services/tree/master/Google/guice-core
will most likely not pass the tests.

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.

3 participants