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

Fix "NSFilenamesPboardType is deprecated" #4221

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

Coeur
Copy link
Collaborator

@Coeur Coeur commented Nov 22, 2022

Another deprecation biting the dust.
This time, NSFilenamesPboardType and NSURLPboardType.

I tested the change is non-breaking by drag&dropping files and urls.

Copy link
Member

@nevack nevack left a comment

Choose a reason for hiding this comment

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

LGTM

@Coeur I kindly ask you to split changes like this in the future.
Maybe not separate PRs, but commits will do.
Most of this PR are typing improvements that are not related to the fix.
It will be easier to review commit by commit.
Thanks!

@Coeur
Copy link
Collaborator Author

Coeur commented Nov 22, 2022

OK for multi commits next time.

Here it was to highlights that, while pasteboard.propertyListForType: was returning an array of strings of filepaths, pasteboard.readObjectsForClasses:options: is returning an array of URLs of filepaths.

@ckerr ckerr force-pushed the coeur/NSPasteboardType branch 2 times, most recently from 47bc1cb to 057a1e7 Compare November 23, 2022 15:14
@ckerr ckerr merged commit 3ecf0df into transmission:main Nov 23, 2022
@Coeur Coeur deleted the coeur/NSPasteboardType branch November 24, 2022 05:41
@ckerr
Copy link
Member

ckerr commented Nov 26, 2022

Notes: Fixed macOS API deprecation warnings.

ckerr added a commit that referenced this pull request Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants