I think if someone has the ability to fix the bug (which inherently makes them a Wine developer) they have the ability to look up SHSTOCKICONID values. It hardly takes expertise to do so, just looking at the code.
There is certainly a wide spectrum of capability for wine developer.
That's not what I'm asking though; I'm asking why you need to change the line with the FIXME
Because the flags value is now partially handled. So changing the fixme to represent this made sense to me.
Are there still necessary changes, or is this sufficient?