On Sun, 16 Feb 2025, LIU Hao wrote:


--

In general, I think this is safe - however, are some of these headers ones that are autogenerated from IDL sources? In that case, we should fix the IDL (and possibly the upstream version in Wine) first.

In the following files, there seem to be changes that aren't just removing trailing commas: bdaiface_enums.h evntprov.h
rasshost.h
tsattrs.h
wbemidl.h

In at least one of them, it seems to be that we're getting rid of spurious CRLF line endings. That's probably fine, but please check these cases, and I'd prefer to split them out to a separate commit (no need to send for review, just push such changes). In one case, the whole file is rewritten (with the right kind of newlines?).

In tcpmib.h there's a change to a pseudocode case in a comment - no strong opinion on whether we should leave that be or just update it consistently.

The changes in wiadef.h seem to be in more complex code than the rest; please doublecheck the change that it is right (I would presume so, but it deserves a manual check).

// Martin



_______________________________________________
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to