Re: Changed more malloc to HeapAlloc calls

2008-10-30 Thread Pete Myers
e out? There's obviously something I'm missing! Pete 2008/10/30 James Hawkins <[EMAIL PROTECTED]> > On Wed, Oct 29, 2008 at 7:39 PM, Pete Myers > <[EMAIL PROTECTED]> wrote: > > In line with http://wiki.winehq.org/ReplaceMalloc this is a small patch > that &g

Re: Changed more malloc to HeapAlloc calls

2008-10-30 Thread Pete Myers
> In addition to what James said please split the patch up; one patch per > module. A dll or program is a "module". Sorry, I'm inundated with noobie errors here I think. > Also while you are at it please remove the superfluous casts in the lines > you are changing. HeapAlloc (malloc too) returns

Re: Changed more malloc to HeapAlloc calls

2008-10-30 Thread Pete Myers
2008/10/30 James Hawkins <[EMAIL PROTECTED]> > > On Wed, Oct 29, 2008 at 8:02 PM, Pete Myers > <[EMAIL PROTECTED]> wrote: > > It does match the style of the way that HeapAlloc is used elsewhere in the > > file, though not malloc admittedly. I wasn't s