Re: [PATCH v2 0/5] MR665: win32u: Tweak reading / writing of DEVMODEW into the registry.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
>  query_mode_field( y_resolutionW, dmPelsHeight, DM_PELSHEIGHT );
>  query_mode_field( v_refreshW, dmDisplayFrequency, DM_DISPLAYFREQUENCY );
>  query_mode_field( flagsW, dmDisplayFlags, DM_DISPLAYFLAGS );
> - if (index == ENUM_CURRENT_SETTINGS || index == ENUM_REGISTRY_SETTINGS)
> - {
> -
>    query_mode_field( x_panningW, dmPosition.x, DM_POSITION );
> -
>    query_mode_field( y_panningW, dmPosition.y, DM_POSITION );
> - }
> query_mode_field( orientationW, dmDisplayOrientation, DM_DISPLAYORIENTATION );
> - query_mode_field( fixed_outputW, dmDisplayFixedOutput, 0 );
> - query_mode_field( fixed_outputW, dmDisplayFixedOutput, DM_DISPLAYFIXEDOUTPUT );
> - query_mode_field( x_panningW, dmPosition.x, DM_POSITION );

For read_adapter_mode(). I think it's better to keep the "if (index == ENUM_CURRENT_SETTINGS || index == ENUM_REGISTRY_SETTINGS)" for querying the position, so that unnecessary registry queries can be avoided.