Out-of-memory check is performed by NdrAllocate and so isn't required to 
be done in the unmarshalling functions.

Tests show that NdrPointerFree (and hence NdrFree) don't check whether 
the memory was allocated with NdrAllocate before freeing it. This makes 
sense as servers don't need to use NdrAllocate to allocate memory being 
returned and so this commit caused that memory to be leaked.

NdrAllocate hasn't been changed as the tests show it is correct and it 
appears that the memory list is used to implement the RpcSs memory model.

Add comments to justify why this is the only check that is needed.

These are supposed to point to the original buffer when the types were 
unmarshalled.

Revert "rpcrt4: Try to free every non-stack pointer in PointerFree now that NdrFree does the sanity checking for us.".

This reverts commit 6db648302ce43f625a073426a35a6f40b7869d8b.

Tests show the assumptions in a commit that this commit depends on are 
not valid.

widl: Don't write redundant FC_RP type format out for conformant strings, unless it is a top level variable.

Cache the enum info so that repeated calls to EnumFontFamiles and such
does not have to repeatedly load the font file.

widl: Remove some confusion in how version numbers are packed into an attribute by defining some macros for this purpose.

Remove unneeded includes.

rpcrt4: Try to free every non-stack pointer in PointerFree now that NdrFree does the sanity checking for us.

rpcrt4: Store allocated memory in a singly-linked list to keep track of what we should and shouldn't free in NdrFree.