Re: [PATCH] d2d1: Don't update target dpi if only one SetDpi() argument is zero

[Henri Verbeet]

On 19 January 2016 at 22:43, Nikolay Sivov <nsivov(a)codeweavers.com> wrote:
> @@ -1436,6 +1436,8 @@ static void STDMETHODCALLTYPE d2d_d3d_render_target_SetDpi(ID2D1RenderTarget *if
>          dpi_x = 96.0f;
>          dpi_y = 96.0f;
>      }
> +    else if (dpi_x == 0.0f || dpi_y == 0.0f)
> +        return;

Should this affect rendertarget creation (i.e.,
d2d_d3d_render_target_init()) as well? What about bitmap creation?

[PATCH] d2d1: Don't update target dpi if only one SetDpi() argument is zero

[Nikolay Sivov]

On 20.01.2016 15:06, Henri Verbeet wrote:
> On 19 January 2016 at 22:43, Nikolay Sivov <nsivov(a)codeweavers.com> wrote:
> > @@ -1436,6 +1436,8 @@ static void STDMETHODCALLTYPE d2d_d3d_render_target_SetDpi(ID2D1RenderTarget *if
> >          dpi_x = 96.0f;
> >          dpi_y = 96.0f;
> >      }
> > +    else if (dpi_x == 0.0f || dpi_y == 0.0f)
> > +        return;
>
> Should this affect rendertarget creation (i.e.,
> d2d_d3d_render_target_init()) as well? What about bitmap creation?

Good question, I'll test it.