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

[Nikolay Sivov]

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.