[[ Cleanup ]] MCCacheHandle::GetFilePointer(): false is non-pointer type.
[[ Cleanup ]] MCMultiPartParseHeaderParams(): r_param_count is of non-pointer type.
[[ Cleanup ]] get_variable_ex_utf8(): false is non-pointer type.
[[ Cleanup ]] MCExternalVariable: Avoid double-dereferencing pointers.

Previously, MCExternalVariable's AppendInteger() and AppendCardinal()
methods each cast their 32-bit integer argument to a pointer and
dereferenced it, which was clearly the Wrong Thing on any non-32-bit
platform.

When I looked into this further, I found that both call sites for
these methods passed in values, not pointers, so the previous
behaviour was Obviously Wrong.

If this change causes regressions then we'll need to fix the API to be
64-bit safe and a lot less abusive.
[[ Cleanup ]] MCExternalVariableAppend(): Initialise 16-bit string correctly.

Prevent appending UTF16 strings from crashing the program due to an
incorrectly-initialised variable.
[[ Cleanup ]] MCSystemOpenElevatedProcess(): Preserve string literal const-ness.