Re: UPDATED: video mode changes

[Dimitrie O. Paun]

On April 12, 2003 09:34 pm, Alex Pasadyn wrote:
> +#define _X_FIELD(prefix, bits) if ((fields) & prefix##_##bits) TRACE(""#bits ","); +static void _dump_CDS_flags(DWORD fields) {
> -
>   _X_FIELD(CDS,UPDATEREGISTRY);_X_FIELD(CDS,TEST);_X_FIELD(CDS,FULLSCREEN);
> -
>   _X_FIELD(CDS,GLOBAL);_X_FIELD(CDS,SET_PRIMARY);_X_FIELD(CDS,RESET); 
> -
>   _X_FIELD(CDS,SETRECT);_X_FIELD(CDS,NORESET);
>
> +}
> +static void _dump_DM_fields(DWORD fields) {
> -
>   _X_FIELD(DM,BITSPERPEL);_X_FIELD(DM,PELSWIDTH);_X_FIELD(DM,PELSHEIGHT);
> -
>   _X_FIELD(DM,DISPLAYFLAGS);_X_FIELD(DM,DISPLAYFREQUENCY);_X_FIELD(DM,POSITION);
>
> +}
> +#undef _X_FIELD

I still think these should return a string as I described in my
previous email...

Also, there is no need for such tests:
> -
>    if (TRACE_ON(x11drv))

The TRACE will take care of things nicely.