Skip to content

specs/XamlBindingHelper_Spec.md#11022

Open
rashmi-thakurr wants to merge 2 commits intomainfrom
user/rashmithakur/xamlbindinghelper-spec
Open

specs/XamlBindingHelper_Spec.md#11022
rashmi-thakurr wants to merge 2 commits intomainfrom
user/rashmithakur/xamlbindinghelper-spec

Conversation

@rashmi-thakurr
Copy link
Copy Markdown

Fixes

PR Type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Description

Current Behavior

New Behavior

Motivation and Context

How Has This Been Tested?

  • I have performed a self-review of my own code
  • I have added tests to cover my changes

Screenshots (if appropriate):

@microsoft-github-policy-service microsoft-github-policy-service bot added the needs-triage Issue needs to be triaged by the area owners label Mar 11, 2026

### Remarks

* `dependencyObject` must not be `null`; an `E_POINTER` error is returned if it is `null`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Return type seems to be void

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Updated it to - If dependencyObject or propertyToSet is null, an exception is thrown by the WinRT layer from the underlying E_POINTER failure

// Existing members omitted for brevity

[contract(Microsoft.UI.Xaml.WinUIContract, 10)]
[static_name("Microsoft.UI.Xaml.ISetterStatics2")]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why ISetterStatics2 is required? I think it can be ISetterStatics only

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

ISetterStatics2 is created by codegen likely because ValueProperty is in new contract.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-triage Issue needs to be triaged by the area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants