amerkoleci / Vortice.Win32

Low level .NET bindings for Win32 Api.
MIT License
49 stars 2 forks source link

Remove autoproperties from Color4 and Viewport types #1

Closed Sergio0694 closed 1 year ago

Sergio0694 commented 1 year ago

This fixes some build errors on .NET Native (UWP). Same changes as in https://github.com/Sergio0694/ComputeSharp/pull/377.

amerkoleci commented 1 year ago

Would public readonly field work as well? For example: public readonly float Width:

Sergio0694 commented 1 year ago

"Would public readonly field work as well?"

Yup, that'd also work fine! Only autoproperties are an issue in marshalled types. I kept them in this PR and didn't switch to fields as I didn't want to introduce any breaking changes on a public API, as I didn't know whether you'd have wanted that 😄