Re: JanitorialProjects/ReplaceMalloc: remove malloc and free from dlls folder

2010-12-31 Thread Dan Kegel
On Fri, Dec 31, 2010 at 3:48 PM, Matteo Bruni wrote: > Maybe it's just time to remove the "replace malloc with HeapAlloc" > task from the wiki? Or, at least, add a note to explain that we expect > the current Wine code to be "compliant". Done.

Re: JanitorialProjects/ReplaceMalloc: remove malloc and free from dlls folder

2010-12-31 Thread Matteo Bruni
2010/12/30 Dan Kegel : > Yegor wrote: >> it's my first contribution to wine... > > Welcome! > >> so I wanted to start with some simple task. > > Changing memory allocation might not be simple. > > Your patch touches two different DLLs. > You should probably split this patch up, one patch per dll. >

re: JanitorialProjects/ReplaceMalloc: remove malloc and free from dlls folder

2010-12-30 Thread Dan Kegel
Yegor wrote: > it's my first contribution to wine... Welcome! > so I wanted to start with some simple task. Changing memory allocation might not be simple. Your patch touches two different DLLs. You should probably split this patch up, one patch per dll. Also, the fact that your change to d3dc