`windows_bindgen` should make use of `Option<NonNull<T>>` instead of `Option<*mut/const T>` for `ptr` params.

[chernoivanenkoofficial]

Suggestion

Currently, a lot of bindings for optional in or out pointer parameters of bindings are using Option around the possibly null pointer, which:

1. Makes a param size larger then a size of a ptr,
2. Requires casting to raw pointer to be a real None check and not a no-op.

Making binding rely on Option<NonNull<T>> should be a general and quite easy improvement (though, it will not allow to express if the param can be mutated by bound function). Other then this little drawback, are there any other reasons not to opt for this kind of binding generating?

[Adrien-Bodineau]

NonNull would only work for *mut p as stated by the documentation.
but for strictly out parameters that would be a good idea I think

[kennykerr]

Sorry I haven't acknowledged this issue yet. It's something I've been mulling over in the back of my head, but haven't yet had enough time to really consider what the right solution is.

[github-actions]

This issue is stale because it has been open 10 days with no activity. Remove stale label or comment or this will be closed in 5 days.