Gabriel Ivăncescu (@insn) commented about dlls/shell32/shellitem.c:
WINE_DEFAULT_DEBUG_CHANNEL(shell);
-typedef struct _ShellItem { +typedef struct { IShellItem2 IShellItem2_iface; LONG ref; LPITEMIDLIST pidl; IPersistIDList IPersistIDList_iface; IShellItemImageFactory IShellItemImageFactory_iface; -} ShellItem; +} ShellItemImpl;
In my opinion it's probably better to use something like "shell_item" since you're changing it and that has zero chance to conflict due to being in snake case. You could also get rid of the typedef if you want to make it explicit and use "struct shell_item" when declaring types of it instead.
Typically rest of wine code which uses snake case tends to either typedef with a `_t` suffix at the end, or use `struct type_name` with no typedef. (`_t` has slight problem that it can cause conflicts with POSIX)