Re: [PATCH 2/2] wined3d: Turn off message filtering temporarily in the Reset() method to allow certain messages (e.g. WM_ACTIVATEAPP) through.

[Henri Verbeet]

On 2 May 2011 13:18, John Edmonds  wrote:
> Toki Tori and RUSH both depend on receing WM_ACTIVATEAPP before they will start. These messages are currently getting filtered out.
> This resolves bug #25640.

It doesn't look right to me to filter messages during
IWineD3DDevice_ReleaseFocusWindow() but not
IWineD3DDevice_AcquireFocusWindow(). Maybe it just shouldn't filter at
all.

[PATCH 2/2] wined3d: Turn off message filtering temporarily in the Reset() method to allow certain messages (e.g. WM_ACTIVATEAPP) through.

[John Edmonds]

Are you suggesting that we turn off message filtering altogether or just for
the Reset function?

I would like to add that I did disable all message filtering (by commenting
out the code in Wine's wndproc) and the games I was testing worked fine.
However, I did not investigate the effects of that change very thoroughly.
On May 2, 2011 7:52 AM, "Henri Verbeet"  wrote:
> On 2 May 2011 13:18, John Edmonds  wrote:
> > Toki Tori and RUSH both depend on receing WM_ACTIVATEAPP before they will

start. These messages are currently getting filtered out.
> > This resolves bug #25640.
>
> It doesn't look right to me to filter messages during
> IWineD3DDevice_ReleaseFocusWindow() but not
> IWineD3DDevice_AcquireFocusWindow(). Maybe it just shouldn't filter at
> all.

[PATCH 2/2] wined3d: Turn off message filtering temporarily in the Reset() method to allow certain messages (e.g. WM_ACTIVATEAPP) through.

[Henri Verbeet]

On 2 May 2011 23:28, John Edmonds  wrote:
> Are you suggesting that we turn off message filtering altogether or just for
> the Reset function?

Just Reset().