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

CFURL and CFAttributedString patches for TextEdit #6

Merged
merged 2 commits into from
Feb 3, 2024

Conversation

CKegel
Copy link

@CKegel CKegel commented Apr 12, 2023

Adds Type Identification to CFURl(and by extension NSURL) using the LaunchServices UTTypeCreatePreferredIdentifierForTag(...) API.

Copy link
Member

@bugaevc bugaevc left a comment

Choose a reason for hiding this comment

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

Nice work — nitpicking as promised 😀

NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Show resolved Hide resolved
@CKegel
Copy link
Author

CKegel commented May 1, 2023

The only thing blocking this PR is the discussion regarding whether the symbols from LaunchServices should be loaded and unloaded for each function call or loaded once statically, and whether that loading should have global or local scope. I'd like to examine what Apple does, however I do not have access to a Mac and am not sure how to do so. I think I've handled everything except for squashing the commits(I'm waiting to do this until I verify the code is correct)

@CKegel CKegel requested a review from bugaevc May 26, 2023 15:30
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
@CKegel CKegel force-pushed the Add_UTIs branch 2 times, most recently from 38bd89e to 6107bdb Compare May 28, 2023 22:17
@CKegel CKegel requested a review from bugaevc June 23, 2023 01:33
@CKegel CKegel changed the title Add kCFURLTypeIdentifierKey to CFURL(and NSURL) CFURL and CFAttributedString patches for TextEdit Oct 20, 2023
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
NSURL.m Outdated Show resolved Hide resolved
Copy link
Member

@bugaevc bugaevc left a comment

Choose a reason for hiding this comment

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

I can't see anything wrong with this anymore, other than minor stylistic nitpicks that are, at this point, not worth obsessing over. So given that, and that Ariel has approved the change as well, let's go 🚀

@CuriousTommy CuriousTommy merged commit ef09be6 into darlinghq:master Feb 3, 2024
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.

4 participants