Re: [PATCH 3/4] localspl: Add support for monitors providing the MONITOR2 interface.

[Huw Davies]

On Fri, Aug 30, 2019 at 04:06:34PM +0800, Dmitry Timoshkov wrote:
> Signed-off-by: Dmitry Timoshkov
>
> dlls/localspl/provider.c | 149 +++++++++++++++++++++++++++++----------
>  1 file changed, 111 insertions(+), 38 deletions(-)
>
> diff --git a/dlls/localspl/provider.c b/dlls/localspl/provider.c
> index 8e5e3428d5..2b3f9bac6e 100644
> --- a/dlls/localspl/provider.c
> +++ b/dlls/localspl/provider.c
> @@ -66,9 +66,9 @@ typedef struct {
>      LPWSTR          dllname;
>      PMONITORUI      monitorUI;
>      LPMONITOR       monitor;
> - LPMONITOR2      monitor2;
> HMODULE         hdll;
> DWORD           refcount;
> - DWORD           dwMonitorSize;
>
> } monitor_t;

Hi Dmitry,

Is there a reason why we can't simply replace the MONITOR struct with
a MONITOR2 struct?   It would require a bit more work at initialisation
but then calling the functions would be rather simpler.

Thanks,
Huw.