Declaring Non-Nullable Parameters

[kennykerr]

This is a description of the NotNull attribute. Subsequent work includes:

• Updating the WinRT metadata to include the new attribute and adding MIDLRT tests for it.
• Updating C#/WinRT to apply the new attribute
• Updating Rust/WinRT to apply the new attribute

[Marv51]

Hi everyone, may I ask what happened to this pull request? Looks like it was approved by three people and then closed without merging?
@kennykerr?

[kennykerr]

I think it was deemed impractical to go back and annotate the many existing APIs that are known to be non-null. I have since come up with a solution for Rust that lets me deal with nullable parameters elegantly, so I no longer have a pressing need for this.

[dotMorten]

I have since come up with a solution for Rust that lets me deal with nullable parameters elegantly

@kennykerr but what about C#? We still have a pressing need for this.

it was deemed impractical to go back and annotate the many existing API

Impractical or a lot of work? Doing it for .NET was impractical and a lot of work, but it was done.

[kennykerr]

Those things, and somebody actually needs to do the work. 😜 It's just a considerable amount of work across a much larger code base by not one but many disparate teams who are no longer actively maintaining many of those APIs.

[dlech]

Could the work be automated (e.g. with static analysis to detect nullability)? Could someone external to Microsoft do the work?

[kennykerr]

No, but you can apply here: https://careers.microsoft.com/ 😉