[PATCH 1/7] msctf: Add langbarmgr.c

Nikolay Sivov bunglehead at gmail.com
Thu Feb 11 11:22:49 CST 2010


On 2/11/2010 19:51, Justin Chevrier wrote:
> ---
>   dlls/msctf/langbarmgr.c |  193 +++++++++++++++++++++++++++++++++++++++++++++++
>   1 files changed, 193 insertions(+), 0 deletions(-)
>   create mode 100644 dlls/msctf/langbarmgr.c
>
>    

> +    This = HeapAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,sizeof(LangBarMgr));
>
>    
You don't need HEAP_ZERO_MEMORY here, cause you will set all fields anyway.

Also it looks like you specially split to several patches to have only 
one file per patch, that's not needed.




More information about the wine-devel mailing list