On 02/03/2018 04:03 PM, Nikolay Sivov wrote:
That's obviously correct, but saxreader.c and schema.c are still using HeapAlloc() directly. I think it makes sense to fix all of it in one go.For me that's a separate patch. I'm ignoring the HeapAlloc
transformation at the moment.
I'm focusing now on removing the local wrappers. Once I'm done with those I'll shift my attention to the heap_strdup* wrappers. Then I'm still thinking to give the infallible heap_alloc variant another go. Only after that is settled it makes sense for me to go over all the HeapAlloc() and see how to transform them. But of course everybody else doesn't have to follow this and do the HeapAlloc ==> heap_alloc change as they see fit.
bye michael