Re: [PATCH 1/3] user32: Free window text to prevent leak in case whenWM_NCDESTROY doesn't get to default procedure

2009-11-16 Thread Nikolay Sivov
The only case why I left it was that I thought about app that directly
sends this message for some insane reason. I just don't want to touch
default handler.

On 11/16/09, Dmitry Timoshkov  wrote:
> "Nikolay Sivov"  wrote:
>
>> This fixes a leak spotted by valgrind.
>>
>> Changelog:
>>- free window text to prevent leak in case when WM_NCDESTROY
>> doesn't get to default procedure
>
> Then WM_NCDESTROY handler should be removed from DefWindowProc
> to avoid a confusing duplication.
>
> --
> Dmitry.
>




Re: [PATCH 1/3] user32: Free window text to prevent leak in case whenWM_NCDESTROY doesn't get to default procedure

2009-11-15 Thread Dmitry Timoshkov

"Nikolay Sivov"  wrote:


This fixes a leak spotted by valgrind.

Changelog:
   - free window text to prevent leak in case when WM_NCDESTROY
doesn't get to default procedure


Then WM_NCDESTROY handler should be removed from DefWindowProc
to avoid a confusing duplication.

--
Dmitry.