-
Notifications
You must be signed in to change notification settings - Fork 119
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
More and more WinForms enums #1245
Conversation
@chenss3 What is the merge conflict on? Apparently I cannot see what is causing it. |
This branch doesn't contain the .gitattributes change. Sync with main and see if that helps. The conflict is in the changes file. |
Should be fixed now! |
When you synced with main, did the changes file merge fine? |
When I synced with main that time, I still had to manually resolve the merge conflicts in the changes file. |
scripts/ChangesSinceLastRelease.txt
Outdated
Windows.Win32.System.Ole.Apis.OleLoadPicture : lplpvObj : [In,Out] => [Out] | ||
Windows.Win32.System.Ole.Apis.OleLoadPictureEx : lplpvObj : [In,Out] => [Out] | ||
Windows.Win32.System.Ole.Apis.OleLoadPicturePath : ppvRet : [In,Out] => [Out] | ||
Windows.Win32.System.Ole.Apis.OleTranslateColor : lpcolorref : [In,Out] => [Out] | ||
# UIA_ enums | ||
Windows.Win32.System.SystemServices.UIA_ANNOTATIONTYPE added |
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.
APIs in the SystemServices namespace indicates a bug. Even though the UIA changes went in with another PR, please update enums.json to move anything in SystemServices over to a more relevant namespace like Accessibility.
Either add a usage so the enum follows the namespace of the APIs, or add an explicit namespace definition.
Fixes: #1092
Fixes: #1076
Fixes: #1121
Fixes: #1086
Fixes: #1089
Metadata Diff: