23 Jul
2025
23 Jul
'25
6:40 p.m.
Why bother with these? I don't think it's that important to have a string printed, we can always look it up later.
I would defend these to say it is not always a wine developer attempting to read the fixme's to see why something went wrong.
It was not clear to me that an icon was missing when trying to get this issue resolved.
Why change the message?
Because it is not necessarily true that if there are any flags, that they are unhandled. If you provide just `SHGSI_ICON`, we do set hIcon. This was how this was handled in another function with partial flags handling.
Why add braces?
This is like the 7th round of changes at this point, I didn't notice after the last rebase. I'll undo it.