[vlc-devel] [PATCH 05/20] xcb/x11: use the picture placement from the core on open

Rémi Denis-Courmont remi at remlab.net
Tue Aug 25 15:42:02 CEST 2020


Le tiistaina 25. elokuuta 2020, 10.29.53 EEST Steve Lhomme a écrit :
> No need to compute it ourselves.
> ---
>  modules/video_output/xcb/x11.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/modules/video_output/xcb/x11.c b/modules/video_output/xcb/x11.c
> index c0f2e18b039..52130b0ad8d 100644
> --- a/modules/video_output/xcb/x11.c
> +++ b/modules/video_output/xcb/x11.c
> @@ -309,14 +309,13 @@ static int Open (vout_display_t *vd, const
> vout_display_cfg_t *cfg, /* XCB_CW_COLORMAP */
>          cmap,
>      };
> -    vout_display_place_t place;
> +    const vout_display_place_t *place = vd->place;
> 
> -    vout_display_PlacePicture(&place, &vd->source, cfg);
>      sys->window = xcb_generate_id (conn);
>      sys->gc = xcb_generate_id (conn);
> 
>      xcb_create_window(conn, sys->depth, sys->window,
> cfg->window->handle.xid, -        place.x, place.y, place.width,
> place.height, 0,
> +        place->x, place->y, place->width, place->height, 0,
>          XCB_WINDOW_CLASS_INPUT_OUTPUT, vid, mask, values);
>      xcb_map_window(conn, sys->window);
>      /* Create graphic context (I wonder why the heck do we need this) */

OK

-- 
レミ・デニ-クールモン
http://www.remlab.net/





More information about the vlc-devel mailing list