Re: [ros-svn] [ion] 16727: - Message Queue Fix, resolves a number of application regressions (Total Commander works again, for example). Patch by Hartmut Birr. To be commited into 0.2.7 after more testing.

[Hartmut Birr]

[e-mail] wrote:
> Message Queue Fix, resolves a number of application regressions (Total Commander works again, for example). Patch by Hartmut Birr. To be commited into 0.2.7 after more testing.
>
> Updated files:
> trunk/reactos/subsys/win32k/ntuser/window.c

Hi,

this fix wasn't for commiting, because it doesn't solve the real
problem. The destroying of the message queues is intitiated by calling
Win32kThreadCallback which calls MsqDestroyMessageQueue. After this
call, it shouldn't exist anymore a  window for this thread.

- Hartmut

[Alex Ionescu]

Hartmut Birr wrote:
> [e-mail] wrote:
> > Message Queue Fix, resolves a number of application regressions (Total Commander works again, for example). Patch by Hartmut Birr. To be commited into 0.2.7 after more testing.
> >
> > Updated files:
> > trunk/reactos/subsys/win32k/ntuser/window.c
>
> Hi,
>
> this fix wasn't for commiting, because it doesn't solve the real
> problem. The destroying of the message queues is intitiated by calling
> Win32kThreadCallback which calls MsqDestroyMessageQueue. After this
> call, it shouldn't exist anymore a  window for this thread.
> Hartmut

I know it's a hack but it's all we've got to fix that blocker bug... 
does it break anything? If not I don't see anything wrong with having it 
in tempoarily.

Best regards,
Alex Ionescu