Skip to content

Comments

Stubs and symbols for iTerm2#39

Merged
CuriousTommy merged 7 commits intodarlinghq:masterfrom
sirnacnud:iterm2
Jun 19, 2024
Merged

Stubs and symbols for iTerm2#39
CuriousTommy merged 7 commits intodarlinghq:masterfrom
sirnacnud:iterm2

Conversation

@sirnacnud
Copy link

I attempted to get iTerm2 running under darling, these were the symbols/stubs I needed to implement in Cocotron. While I didn't manage to get iTerm2 fully running, I thought to push these up anyways.

Fixes specific missing symbol mentioned in: darlinghq/darling#1350

Stubs/Symbols Added

  • AppKit: NSFontCollection, NSCandidateListTouchBarItem, NSPasteboardNameFind, NSPasteboardNameFont, NSScrubberSelectionStyle, NSScrubberItemView
  • CoreData: NSSQLiteManualVacuumOption
  • CoreGraphics: kCGWindowName, kCGWindowOwnerName

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.

Other then the above issues, the code looks good to me!

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.

Forgot to check the @interface part, just need one more change.

I'm currently doing a build on my machine to make sure nothing breaks.


#import <AppKit/NSTouchBarItem.h>

@interface NSCandidateListTouchBarItem<CandidateType> : NSTouchBarItem

Choose a reason for hiding this comment

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

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 0baee1d into darlinghq:master Jun 19, 2024
@sirnacnud sirnacnud deleted the iterm2 branch June 19, 2024 16:48
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