Re: winedbg: Only add ContextFlags which are defined

[Dmitry Timoshkov]

André Hentschel  wrote:
> CONTROL and INTEGER are standard defines, but ia64 and ARM don't define CONTEXT_FLOATING_POINT

ia64 does define CONTEXT_FLOATING_POINT, ARM probably should also do.

winedbg: Only add ContextFlags which are defined

[André Hentschel]

Am 09.09.2010 11:07, schrieb Dmitry Timoshkov:
> André Hentschel  wrote:
> > CONTROL and INTEGER are standard defines, but ia64 and ARM don't define CONTEXT_FLOATING_POINT
>
> ia64 does define CONTEXT_FLOATING_POINT, ARM probably should also do.

A standard ARM CPU doesn't has a FPU, so i guess that don't make sense.
I didn't find that define for ia64, so maybe i have missed something, sry.

winedbg: Only add ContextFlags which are defined

[Dmitry Timoshkov]

André Hentschel  wrote:
> > > CONTROL and INTEGER are standard defines, but ia64 and ARM don't define CONTEXT_FLOATING_POINT
> >
> > ia64 does define CONTEXT_FLOATING_POINT, ARM probably should also do.
>
> A standard ARM CPU doesn't has a FPU, so i guess that don't make sense.

Probably, but Wine is using float internally, and doesn't make any effort
to avoid that. Defining CONTEXT_FLOATING_POINT to 0 on ARM is another solution.

winedbg: Only add ContextFlags which are defined

[André Hentschel]

Am 09.09.2010 12:34, schrieb Dmitry Timoshkov:
> André Hentschel  wrote:
> > > > CONTROL and INTEGER are standard defines, but ia64 and ARM don't define CONTEXT_FLOATING_POINT
> > >
> > > ia64 does define CONTEXT_FLOATING_POINT, ARM probably should also do.
> >
> > A standard ARM CPU doesn't has a FPU, so i guess that don't make sense.
>
> Probably, but Wine is using float internally, and doesn't make any effort
> to avoid that. Defining CONTEXT_FLOATING_POINT to 0 on ARM is another solution.

Just found the function get_server_context_flags in ntdll where the same as in my patch is done...