On 16 February 2018 at 11:54, Gerd Hoffmann <[email protected]> wrote:
> From: Anatoly Trosinenko <[email protected]>
>
> The documentation on SDL_RenderPresent function states that
> "the backbuffer should be considered invalidated after each present",
> so copy the entire texture on each redraw.
>
> On the other hand, SDL_UpdateTexture function is described as
> "fairly slow function", so restrict it to just the changed pixels.
>
> Also added SDL_RenderClear call, as suggested in the documentation
> page on SDL_RenderPresent.
>
> Signed-off-by: Anatoly Trosinenko <[email protected]>
> Message-id: [email protected]
> Signed-off-by: Gerd Hoffmann <[email protected]>
> ---
> ui/sdl2-2d.c | 21 ++++++---------------
> 1 file changed, 6 insertions(+), 15 deletions(-)
>
> diff --git a/ui/sdl2-2d.c b/ui/sdl2-2d.c
> index 8ab68d67b9..1f34817bae 100644
> --- a/ui/sdl2-2d.c
> +++ b/ui/sdl2-2d.c
> @@ -36,6 +36,8 @@ void sdl2_2d_update(DisplayChangeListener *dcl,
> struct sdl2_console *scon = container_of(dcl, struct sdl2_console, dcl);
> DisplaySurface *surf = qemu_console_surface(dcl->con);
> SDL_Rect rect;
> + size_t surface_data_offset = surface_bytes_per_pixel(surf) * x +
> + surface_stride(surf) * y;
>
> assert(!scon->opengl);
>
Hi. Coverity points out (CID 1390598) that this change adds
a use of 'surf' before the "if (!surf) { return; }" check.
I suspect the calculation of surface_data_offset should be
done lower in the function, after all the early-exit checks.
thanks
-- PMM
