> LocalAlloc is deprecated. > So use HeapAlloc instead, once LocalAlloc is an overhead wrapper to > HeapAlloc.
These functions you are patching are not in performance-sensitive code, so I doubt this makes any difference performance wise. I doubt Microsoft will ever remove these deprecated functions, given its history of backwards compatibility, so from that perspective this change does not achieve anything either.
If users don't adapt to the new API, the old one will never really expire.
If you were proposing to change how palloc() allocates memory, that would be quite different and perhaps useful, as long as a benchmark accompanies the patch.
This is irrelevant to the discussion.
Neither the patch nor the thread deals with palloc.