On Tue May 9 15:57:42 2023 +0000, Zebediah Figura wrote:
I think usual Wine style in new code is to put opening braces on their own line, to match the close braces.
On top of that, having `Add = 0,`, which is identical to the MS headers and unnecessary. This is done in other places as well.
My suggestion is to either remove it, or do the rest as: ``` Add = 0, Remove = 1, ``` or even just: ``` Add, Remove, ```
Echoing back to Zeb's original point, a lot of this looks virtually identical to the MS headers. That's the main issue here, and likely why it has yet to be merged.