Robert Shearman wrote:
Christian Costa wrote:
Hi Michael,
This is typically the kind of patches that breaks everything.
It cannot break anything.
The code you find useless, is often use in lazy allocation or error handling. I doubt the current wine code is so bad designed even if there is some cases where the test is useless. For example in memallocator.c, this is wrong because you can release the allocator even before allocating the memory with the SetProperties method.
I think you are a little confused. HeapFree can be called with a NULL pointer. On Win9x, this causes an error to be set, but from Windows 2000 onwards and Wine, it is allowed and is not an error.
Ok! I didn't know that. :-( I feel better now. :-)
Looking at your other patches, there is some changes that seems suspect to me (in winmm and directx).
The new code is perfectly correct, and is cleaner.
Yes it is. :-) It's true that if this code was so bad, Alexandre would have not accepted it.
Bye, Christian