-
Notifications
You must be signed in to change notification settings - Fork 976
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
Convert ITextRangeProvider/ITextProvider to Cswin32 #10022
Conversation
src/System.Windows.Forms.Primitives/src/System/Windows/Forms/Automation/UiaTextProvider.cs
Outdated
Show resolved
Hide resolved
src/System.Windows.Forms.Primitives/src/Interop/OleAut32/Interop.VARIANT.cs
Outdated
Show resolved
Hide resolved
src/System.Windows.Forms.Primitives/src/Interop/OleAut32/Interop.VARIANT.cs
Outdated
Show resolved
Hide resolved
src/System.Windows.Forms.Primitives/src/System/Windows/Forms/Automation/UiaTextRange.cs
Outdated
Show resolved
Hide resolved
…, return SafeArrayScope
...tem.Windows.Forms.Primitives/tests/UnitTests/Windows/Win32/System/Com/SafeArrayScopeTests.cs
Outdated
Show resolved
Hide resolved
src/System.Windows.Forms.Primitives/src/Windows/Win32/System/Com/SafeArrayScope.cs
Outdated
Show resolved
Hide resolved
src/System.Windows.Forms.Primitives/src/System/Windows/Forms/Automation/UiaTextRange.cs
Outdated
Show resolved
Hide resolved
- Remove UiaTextProvider2 - Add safety checks for SafeArrayScope - Add new private constructor for UiaTextRange - Seal internal classes that are not derived from - Add tests
} | ||
} | ||
|
||
@interface = default; | ||
return false; | ||
} | ||
|
||
#pragma warning disable CS1574 // XML comment has cref attribute 'TryGetObjectForIUnknown{TObject}(IUnknown*, bool, out TObject?)' that could not be resolved |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does Intellisense work for the method or not? If it doesn't then we should figure out how to fix this or manually add the comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intellisense can find the method, but for some reason it is not resolving the cref. I can look into more how this can be resolved or just add the comments manually if nothing else.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work!
Related: #9795
ITextProvider
,ITextProvider2
,ITextRangeProvider
to Cswin32SafeArrayScope
to be able to receiveSAFEARRAY*
(similar toComScope
pattern)ComSafeArrayScope
for better usability when we need aSAFEARRAY
that contains COM interfaces (SafeArrayScope<nint>
)VARIANT
anddouble
UiaPoint
andPoint
Microsoft Reviewers: Open in CodeFlow