Skip to content

Expose internal pointer to AImage#470

Open
ShootingKing-AM wants to merge 2 commits into
rust-mobile:masterfrom
ShootingKing-AM:expose-internal-ptr
Open

Expose internal pointer to AImage#470
ShootingKing-AM wants to merge 2 commits into
rust-mobile:masterfrom
ShootingKing-AM:expose-internal-ptr

Conversation

@ShootingKing-AM
Copy link
Copy Markdown

  • needed for doing some direct operation on AImage ptr (maybe unsafe)
  • but anyways i think the library need not enforce AImage ptr to be hidden
- needed for doing some direct operation on AImage ptr (maybe unsafe)
- but anyways i think the library need not enforce AImage ptr to be hidden
@ShootingKing-AM ShootingKing-AM marked this pull request as draft February 28, 2024 14:51
@ShootingKing-AM ShootingKing-AM marked this pull request as ready for review February 28, 2024 14:54
@MarijnS95
Copy link
Copy Markdown
Member

Before doing this, I'd like to consistenize the current split between fn as_ptr(&self) -> *mut T versus fn ptr(&self) -> NonNull<T>, as well as the documentation comments and safety constraints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants