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

Use Core Foundation crate for CF types instead of using Id<_> #27

Merged
merged 84 commits into from
Oct 22, 2024

Conversation

1313
Copy link
Collaborator

@1313 1313 commented Dec 29, 2023

CF provides a better way to interopt and better manage memory with ref counts using the correct retain/release lifecycles.

See: https://github.com/servo/core-foundation-rs
And: https://docs.rs/core-foundation/0.9.4/core_foundation/macro.declare_TCFType.html

@1313 1313 force-pushed the main branch 4 times, most recently from e2668bf to 86bc2e8 Compare January 16, 2024 08:34
1313 added a commit that referenced this pull request Jan 23, 2024
1313 added 10 commits January 23, 2024 22:05
* main: (21 commits)
  Update CHANGELOG.md
  v0.2.5
  fix: Fix wrong config setting PreservesAspectRatio
  v0.2.4
  Update CHANGELOG.md
  Update CHANGELOG.md
  Update CHANGELOG.md
  fix: #34 Add additional configuration settings, Awaiting #27 for the rest
  fix: #33 Reexport SCFrameStatus SYS-type
  Update rust.yml
  CI
  Update rust.yml
  Update rust.yml
  Update rust.yml
  v0.2.3
  Update README.md
  Update .all-contributorsrc
  docs: add Pranav2612000 as a contributor for code (#32)
  feat: add support for pixel_format and source_rect (#30)
  v0.2.2
  ...
@1313 1313 mentioned this pull request Feb 5, 2024
1313 and others added 6 commits March 19, 2024 21:23
Signed-off-by: Per Johansson <[email protected]>
Signed-off-by: Per Johansson <[email protected]>
Signed-off-by: Per Johansson <[email protected]>
Signed-off-by: Per Johansson <[email protected]>
@not-jan
Copy link

not-jan commented Jun 17, 2024

If I want to start using this crate, should I be using this PR?

@1313 1313 mentioned this pull request Aug 31, 2024
@bliongosari1
Copy link

bliongosari1 commented Oct 1, 2024

Hi @1313 any chance this PR gets merged soon? Looking to use it in a production environment

#51

also wondering if this is fixed here

@1313
Copy link
Collaborator Author

1313 commented Oct 1, 2024

Most of the core is done. I'm adding audio capture at the moment. But have not tested it a lot so it could come with initial bugs.

Also need to scan it for memory leaks. But using core-foundation as a base it should be a lot easier.

So soon. I could push out an alpha for version 0.3 if you want to start testing it.

Cheers

@bliongosari1
Copy link

Thanks @1313 definitely would like to test it out anytime its ready

1313 added 6 commits October 22, 2024 00:31
* main:
  Update README.md
  Update README.md
  feat: Set sample_rate and channel_count via objc interface (#48)
  fix: pass correct args to content filter init methods (#47)
  update changelog
  feat: add support for shows_cursor (#45)
  fix changelog
  v0.2.7
  v0.2.7
  Fix ShareableContent when permissions are missing (#42)
  remove bad code
@1313 1313 merged commit b3d4d96 into main Oct 22, 2024
2 checks passed
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