StrCoreAlloc* and StrAlloc*Secure

Rename the StrAllocate* functions to StrCoreAlloc* to make it easier to tell them apart from the original StrAlloc* methods.

Add the StrAlloc*Secure methods back as wrappers around the StrCoreAlloc* functions.
I agree. The difference wasn't immediately obvious, nor was the name. StrCore isn't descriptive, though honestly I have no ideas for anything better.

I wonder, though, is there any reason we would want to conditionally obfuscate hidden variables? If there are any hidden variables it seems they should always be obfuscated for logged lines. Seems most of that is in variables.cpp. The rest looks good.