Eliminate casts of the return value of HeapAlloc

Dimitrie O. Paun dpaun at rogers.com
Mon Mar 21 22:35:42 CST 2005


On Mon, Mar 21, 2005 at 02:03:59PM +0100, Francois Gouget wrote:
> On Mon, 21 Mar 2005, Alexandre Julliard wrote:
> [...]
> >Log message:
> >	Mike McCormack <mike at codeweavers.com>
> >	Eliminate casts of the return value of HeapAlloc.
> 
> This could be turned into a pretty easy janitorial the starting point of 
> which would be the output of this command:
> 
> find . -name "*.[ch]" | xargs egrep "\) *Heap(Re)?Alloc"
> (344 lines)
> 
> There are a couple of places where the output of HeapAlloc is being cast 
> to a DWORD but except for those it seems pretty straightforward.

Yeah, it's so straightforward, we should just do it.
However, I don't think we should sed it, most places
require human touch to maintain sane formatting.

I'll send in a patch for windows/*.c shortly.

-- 
Dimi.



More information about the wine-devel mailing list