Re: Add check to SetMenuItemInfo

[Dmitry Timoshkov]

"Mike Hearn"  wrote:
> Fail SetMenuItemInfo if both MFT_BITMAP and MFT_SEPARATOR are
> specified (an api violation).

I thought that Alexandre will not commit this patch, since it's
obviously wrong.
> - if (lpmii->fType & (MFT_BITMAP | MFT_SEPARATOR)) {
> - WARN("fType contains MFT_BITMAP and MFT_SEPARATOR, API violation\n");
> - return FALSE;
> - }

You are testing here whether either MFT_BITMAP or MFT_SEPARATOR is set,
which is perfectly valid to specify one of them. If you want to test
whether both flags are set simultaneously then use something like this:

if ((lpmii->fType & (MFT_BITMAP | MFT_SEPARATOR)) == (MFT_BITMAP | MFT_SEPARATOR))

A conformance test to show the real behaviour is urgently required.