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.

For simple tables cells are represented with tabs, and a table row is
ended at the end of the paragraph, so native richedit controls
substitute spaces for \tab and \par rich text format control words.

The values returned by EM_SETPARAFORMAT and EM_GETPARAFORMAT previously
indicated an error, and the included tests shows that Windows behaves as
documented.