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
[stdlib] Replace Pointer
by UnsafePointer
in stdlib/src/builtin/object.mojo
#2365
[stdlib] Replace Pointer
by UnsafePointer
in stdlib/src/builtin/object.mojo
#2365
Conversation
Signed-off-by: gabrieldemarmiesse <gabrieldemarmiesse@gmail.com>
Pointer
by UnsafePointer
in stdlib/src/builtin/object.mojo
Pointer
by UnsafePointer
in stdlib/src/builtin/object.mojo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice. @JoeLoser I'm not up to speed on the policy for merging, any concerns?
No concerns from me, I'll merge this later today. We're still working out some custom roles for contributors for merging etc going forward. This will change soon. FYI on this Pointer migration work, I was discussing with @gabrieldemarmiesse offline earlier today and he graciously volunteered to help us out. I mentioned what work we've already done internally that isn't released yet too to avoid duplicate work. Thanks again for helping out here - very much appreciated! |
…c/builtin/… (#38375) …object.mojo` (modularml#2365) Builtins imports behave in a weird way, I had to import LegacyPointer in `stdlib/src/python/_cpython.mojo`, I have no explanation for this. I just import what the compiler asks me to import :p See https://discord.com/channels/1087530497313357884/1224434323193594059/1231287603462930452 Signed-off-by: gabrieldemarmiesse <gabrieldemarmiesse@gmail.com> mojo-orig-commit: f75add0 Co-authored-by: Gabriel de Marmiesse <gabrieldemarmiesse@gmail.com> modular-orig-commit: c7136f6cc820bcb6e7bed41c1e149bc509f1ccaf
…c/builtin/… (#38375) …object.mojo` (modularml#2365) Builtins imports behave in a weird way, I had to import LegacyPointer in `stdlib/src/python/_cpython.mojo`, I have no explanation for this. I just import what the compiler asks me to import :p See https://discord.com/channels/1087530497313357884/1224434323193594059/1231287603462930452 Signed-off-by: gabrieldemarmiesse <gabrieldemarmiesse@gmail.com> mojo-orig-commit: f75add0 Co-authored-by: Gabriel de Marmiesse <gabrieldemarmiesse@gmail.com> modular-orig-commit: c7136f6cc820bcb6e7bed41c1e149bc509f1ccaf
…c/builtin/… (#38375) …object.mojo` (modularml#2365) Builtins imports behave in a weird way, I had to import LegacyPointer in `stdlib/src/python/_cpython.mojo`, I have no explanation for this. I just import what the compiler asks me to import :p See https://discord.com/channels/1087530497313357884/1224434323193594059/1231287603462930452 Signed-off-by: gabrieldemarmiesse <gabrieldemarmiesse@gmail.com> mojo-orig-commit: f75add0 Co-authored-by: Gabriel de Marmiesse <gabrieldemarmiesse@gmail.com> MODULAR_ORIG_COMMIT_REV_ID: c7136f6cc820bcb6e7bed41c1e149bc509f1ccaf
…c/builtin/… (#38375) …object.mojo` (#2365) Builtins imports behave in a weird way, I had to import LegacyPointer in `stdlib/src/python/_cpython.mojo`, I have no explanation for this. I just import what the compiler asks me to import :p See https://discord.com/channels/1087530497313357884/1224434323193594059/1231287603462930452 Signed-off-by: gabrieldemarmiesse <gabrieldemarmiesse@gmail.com> mojo-orig-commit: f75add0 Co-authored-by: Gabriel de Marmiesse <gabrieldemarmiesse@gmail.com> MODULAR_ORIG_COMMIT_REV_ID: c7136f6cc820bcb6e7bed41c1e149bc509f1ccaf
Builtins imports behave in a weird way, I had to import LegacyPointer in
stdlib/src/python/_cpython.mojo
, I have no explanation for this. I just import what the compiler asks me to import :pSee https://discord.com/channels/1087530497313357884/1224434323193594059/1231287603462930452