Skip to content

Conversation

@madsmtm
Copy link
Contributor

@madsmtm madsmtm commented Jan 26, 2025

This includes using the new crate objc2-core-graphics, which notably does not have the CustomData helper that core-graphics. This is probably for the better, as it allows us to avoid a double-boxing of the data.

This also:

  • Bumps MSRV to 1.71 (required for extern "C-unwind" in objc2).
  • Uses dep:* syntax (to avoid breaking changes from added/removed feature flags).

This includes using the new crate `objc2-core-graphics`, which notably
does not have the `CustomData` helper that `core-graphics`. This is
probably for the better, as it allows us to avoid a double-boxing of the
data.
Copy link
Member

@complexspaces complexspaces left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for work again! I like how this looks for the most part but had a few questions around the modified unsafe.

Copy link
Member

@complexspaces complexspaces left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now, thanks!

With this update I should also be able to take another stab at iOS support, hopefully soon.

@complexspaces complexspaces merged commit 108cc38 into 1Password:master Feb 13, 2025
11 checks passed
@madsmtm madsmtm deleted the objc2-v6 branch February 13, 2025 19:45
@torokati44
Copy link
Contributor

I would appreciate a release on crates.io with this bump, at your earliest convenience. Thank you! 🫶

@torokati44
Copy link
Contributor

Release please! 🥺 🙏

@complexspaces
Copy link
Member

Released in v3.5.0 on crates.io. Sorry again for the delay.

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