Skip to content

Comments

Add stubs for Xcode 3.0#47

Merged
CuriousTommy merged 8 commits intodarlinghq:masterfrom
sirnacnud:xcode-3.0
Sep 25, 2025
Merged

Add stubs for Xcode 3.0#47
CuriousTommy merged 8 commits intodarlinghq:masterfrom
sirnacnud:xcode-3.0

Conversation

@sirnacnud
Copy link

@sirnacnud sirnacnud commented Sep 13, 2025

These are stubs required by Xcode 3.0. See each commit title for a description of the stubs.

Copy link

@CuriousTommy CuriousTommy left a comment

Choose a reason for hiding this comment

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

Just need a few changes

@sirnacnud
Copy link
Author

@CuriousTommy I have updated all the NSKeyValueBinding constants to use the typedefs and this Foundation change is now required.

I also sorted and grouped the constants like in the Apple header file.

Copy link

@CuriousTommy CuriousTommy left a comment

Choose a reason for hiding this comment

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

Please be careful with how you use Apple's SDK as a reference. There is NEVER a good reason to copy comments from Apple's SDK.

Other then that, I have some minor code changes I need you to do.

@sirnacnud
Copy link
Author

sirnacnud commented Sep 22, 2025

@CuriousTommy Thanks for the feedback. I made two new commits for fixing the line formatting and updating the constant values. The other fixes I squashed into the previous commits.

Copy link

@CuriousTommy CuriousTommy left a comment

Choose a reason for hiding this comment

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

LGTM

@CuriousTommy CuriousTommy merged commit e399c2a into darlinghq:master Sep 25, 2025
@sirnacnud sirnacnud deleted the xcode-3.0 branch September 25, 2025 16:13
CuriousTommy added a commit to darlinghq/darling that referenced this pull request Sep 27, 2025
darlinghq/darling-cocotron#47 | Add stubs for Xcode 3.0
darlinghq/darling-cocotron#48 | Add symbols for BibDesk
darlinghq/darling-corecrypto#12 | Export _cccurve25519_make_key_pair symbol
darlinghq/darling-foundation#20 | Add macros for NS_TYPED_ENUM
darlinghq/darling-foundation#21 | Add symbols for BibDesk
CuriousTommy added a commit to darlinghq/darling that referenced this pull request Sep 30, 2025
darlinghq/darling-cocotron#47 | Add stubs for Xcode 3.0
darlinghq/darling-cocotron#48 | Add symbols for BibDesk
darlinghq/darling-corecrypto#12 | Export _cccurve25519_make_key_pair symbol
darlinghq/darling-foundation#20 | Add macros for NS_TYPED_ENUM
darlinghq/darling-foundation#21 | Add symbols for BibDesk
CuriousTommy added a commit to darlinghq/darling that referenced this pull request Sep 30, 2025
darlinghq/darling-cocotron#47 | Add stubs for Xcode 3.0
darlinghq/darling-cocotron#48 | Add symbols for BibDesk
darlinghq/darling-corecrypto#12 | Export _cccurve25519_make_key_pair symbol
darlinghq/darling-foundation#20 | Add macros for NS_TYPED_ENUM
darlinghq/darling-foundation#21 | Add symbols for BibDesk
darlinghq/darling-foundation#22 | Fix NSSocketPortNameServer header reference
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.

2 participants