Skip to content

Partially replace custom AppKit bindings with icrate's autogenerated bindings#2982

Merged
madsmtm merged 17 commits intomasterfrom
replace-with-icrate-appkit
Dec 23, 2023
Merged

Partially replace custom AppKit bindings with icrate's autogenerated bindings#2982
madsmtm merged 17 commits intomasterfrom
replace-with-icrate-appkit

Conversation

@madsmtm
Copy link
Member

@madsmtm madsmtm commented Jul 28, 2023

Now that icrate contains autogenerated bindings to AppKit, we should use those instead of defining the bindings ourselves (which can be very error prone and tedious).

Companion PR for making a lot of the unsafe we use in icrate safe: madsmtm/objc2#475

  • Tested on all platforms changed

@madsmtm madsmtm added DS - appkit Affects the AppKit/macOS backend S - maintenance Repaying technical debt labels Jul 28, 2023
@madsmtm madsmtm force-pushed the objc2-new-version branch 3 times, most recently from b7950d4 to 5eafe7b Compare July 28, 2023 21:48
Base automatically changed from objc2-new-version to master July 28, 2023 22:33
@madsmtm madsmtm force-pushed the replace-with-icrate-appkit branch from f8d470b to bff3e48 Compare July 28, 2023 22:39
@madsmtm madsmtm mentioned this pull request Aug 24, 2023
2 tasks
@madsmtm madsmtm mentioned this pull request Dec 5, 2023
2 tasks
@madsmtm madsmtm force-pushed the replace-with-icrate-appkit branch from bff3e48 to c5cd797 Compare December 23, 2023 19:42
@madsmtm madsmtm force-pushed the replace-with-icrate-appkit branch from c5cd797 to eac5fd1 Compare December 23, 2023 19:51
@madsmtm madsmtm marked this pull request as ready for review December 23, 2023 19:51
@madsmtm madsmtm changed the title Replace custom AppKit bindings with icrate's autogenerated bindings Partially replace custom AppKit bindings with icrate's autogenerated bindings Dec 23, 2023
@madsmtm madsmtm merged commit 674657e into master Dec 23, 2023
@madsmtm madsmtm deleted the replace-with-icrate-appkit branch December 23, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DS - appkit Affects the AppKit/macOS backend S - maintenance Repaying technical debt

Development

Successfully merging this pull request may close these issues.

1 participant