This is a long-needed cleanup aimed at removing the ddraw_primary,
ddraw_window, ddraw_width and ddraw_height members from
IWineD3DDeviceImpl, which just do not belong there.  Destination
window and screen handling is supposed to be done by swapchains.

It's needed by DispatchEx implementation.

RPC_FC_ERROR_STATUS_T has a value of 0x10 and therefore isn't a
possible value of "flags_type & 0xf" in the switch statement.

header isn't a string and therefore using strncmp doesn't make sense.

It's the number of elements, not the number of bytes.

shell32: Don't assign a result from a function that returns a BOOL to an HRESULT in shellex_get_dataobj.

Otherwise there could be confusion as to what type GetFullPathNameW is
returning.

This makes it more obvious what the code is doing.

This makes it more obvious what the code is doing.

This makes it more obvious what the code is doing.

This makes it more obvious what the code is doing.

This makes it more obvious what the code is doing.

msrle32: Fix uses of arithmetic operators on Boolean types in MSRLE32_CompressRLE4Line and MSRLE32_CompressRLE8Line.

The "extra_byte" variable is used as though it contains an integer
rather than a TRUE/FALSE value so make it into an integer.

Use a conditional operator to get the same result, but improving
readability of the code.

Also compare the result of IOleObject_QueryInterface with S_OK instead
of using not operator so it is more clear what the code is doing.