Rein Klazes wijn@wanadoo.nl writes:
Changelog: dlls/user : menu.c dlls/user/tests : menu.c
- make menu tests pass on Win9x/ME;
- rename the macro IS_STRING_ITEM to IS_STRING_ITEM_ONLY, which better
reflects what it is doing: absence of other item type flags.
You should not check the Windows version, you should only check the actual behavior. Also I think IS_STRING_ITEM is fine, there's no reason to add _ONLY since there's no other macro that would do a less specific test.