It's inefficient since the association will have to be replaced before
the binding can be used to connect to a server.

bind->Assoc could be NULL if the binding hasn't been resolved.

DDraw can draw to the front buffer only, thus there's never a Present
call which could pass this window. Due to that a drawing-independent
method is needed.

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.