diff options
author | Marton Balint <cus@passwd.hu> | 2019-12-31 02:43:49 +0300 |
---|---|---|
committer | Marton Balint <cus@passwd.hu> | 2020-01-24 03:17:14 +0300 |
commit | b7e94adb4e51af75eb5cc813e5f4e790010b86c2 (patch) | |
tree | aa27889c1b86c0f5614c65964d3b2f524c17cfa9 /libavdevice/xcbgrab.c | |
parent | fee4cafbf52f81ffd6ad7ed4fd0a8096f8791886 (diff) |
avdevice/xcbgrab: move some initialization code from pixfmt_from_pixmap_format to create_stream
Signed-off-by: Marton Balint <cus@passwd.hu>
Diffstat (limited to 'libavdevice/xcbgrab.c')
-rw-r--r-- | libavdevice/xcbgrab.c | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/libavdevice/xcbgrab.c b/libavdevice/xcbgrab.c index 113cce71a5..06b486a536 100644 --- a/libavdevice/xcbgrab.c +++ b/libavdevice/xcbgrab.c @@ -487,7 +487,7 @@ static xcb_screen_t *get_screen(const xcb_setup_t *setup, int screen_num) } static int pixfmt_from_pixmap_format(AVFormatContext *s, int depth, - int *pix_fmt) + int *pix_fmt, int *bpp) { XCBGrabContext *c = s->priv_data; const xcb_setup_t *setup = xcb_get_setup(c->conn); @@ -525,14 +525,7 @@ static int pixfmt_from_pixmap_format(AVFormatContext *s, int depth, } if (*pix_fmt) { - c->bpp = fmt->bits_per_pixel; - c->frame_size = c->width * c->height * fmt->bits_per_pixel / 8; -#if CONFIG_LIBXCB_SHM - c->shm_pool = av_buffer_pool_init2(c->frame_size + AV_INPUT_BUFFER_PADDING_SIZE, - c->conn, allocate_shm_buffer, NULL); - if (!c->shm_pool) - return AVERROR(ENOMEM); -#endif + *bpp = fmt->bits_per_pixel; return 0; } @@ -592,9 +585,18 @@ static int create_stream(AVFormatContext *s) st->codecpar->width = c->width; st->codecpar->height = c->height; - ret = pixfmt_from_pixmap_format(s, geo->depth, &st->codecpar->format); - + ret = pixfmt_from_pixmap_format(s, geo->depth, &st->codecpar->format, &c->bpp); free(geo); + if (ret < 0) + return ret; + + c->frame_size = c->width * c->height * c->bpp / 8; +#if CONFIG_LIBXCB_SHM + c->shm_pool = av_buffer_pool_init2(c->frame_size + AV_INPUT_BUFFER_PADDING_SIZE, + c->conn, allocate_shm_buffer, NULL); + if (!c->shm_pool) + return AVERROR(ENOMEM); +#endif return ret; } |