Skip to content

Use shared DataFormats code #10391

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

Merged
merged 2 commits into from
Mar 5, 2025
Merged

Use shared DataFormats code #10391

merged 2 commits into from
Mar 5, 2025

Conversation

JeremyKuhne
Copy link
Member

@JeremyKuhne JeremyKuhne commented Feb 4, 2025

This uses the shared DataFormats code from System.Private.Windows.Core.

Note that locking is handled by the shared code so there is no need to do so when adding the WPF specific types. The only reason they need added is that casing will stick with whatever the first request is. That is, if you ask for "XAML" that will be the string Windows gives back, not "Xaml".

Microsoft Reviewers: Open in CodeFlow

This uses the shared DataFormats code from System.Private.Windows.Core.

Note that locking is handled by the shared code so there is no need to do so when adding the WPF specific types. The only reason they need added is that casing will stick with whatever the first request is. That is, if you ask for "XAML" that will be the string Windows gives back, not "Xaml".
@JeremyKuhne JeremyKuhne requested review from a team as code owners February 4, 2025 04:22
@dotnet-policy-service dotnet-policy-service bot added the PR metadata: Label to tag PRs, to facilitate with triage label Feb 4, 2025
@miloush
Copy link
Contributor

miloush commented Feb 4, 2025

Where can I see the value of PInvokeCore.INK_SERIALIZED_FORMAT?

@JeremyKuhne
Copy link
Member Author

Where can I see the value of PInvokeCore.INK_SERIALIZED_FORMAT?

@miloush You have to go to def in VS, this is all source generated. It comes indirectly from the Windows headers. In this particular case it isn't in the docs that I can find (very unusual), just the headers. The value is "Ink Serialized Format".

Copy link
Member

@Kuldeep-MS Kuldeep-MS left a comment

Choose a reason for hiding this comment

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

Changes looks good to me. Can you please add the reference to the PR which is handling the new common code for dataformats?

@JeremyKuhne
Copy link
Member Author

Changes looks good to me. Can you please add the reference to the PR which is handling the new common code for dataformats?

The primary refactoring was dotnet/winforms#12802. The shared code had a number of other edits past that.

@JeremyKuhne JeremyKuhne merged commit dc44448 into dotnet:main Mar 5, 2025
8 checks passed
@JeremyKuhne JeremyKuhne deleted the sharedclip branch March 5, 2025 19:47
@github-actions github-actions bot locked and limited conversation to collaborators Apr 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants