diff options
| author | wm4 <wm4@nowhere> | 2012-11-10 02:02:24 +0100 |
|---|---|---|
| committer | wm4 <wm4@nowhere> | 2013-01-13 17:39:32 +0100 |
| commit | d77d9fb933e0e7dc34de02f3d0c13791ddd80114 (patch) | |
| tree | ca45e0f9d753e7ee7b5bd142eb2aed1638bf391c /video/out/vo_sdl.c | |
| parent | 1568161aadf24ee3a6d982612b7380f8b1dc4a58 (diff) | |
| download | mpv-d77d9fb933e0e7dc34de02f3d0c13791ddd80114.tar.bz2 mpv-d77d9fb933e0e7dc34de02f3d0c13791ddd80114.tar.xz | |
mp_image: require using mp_image_set_size() for setting w/h
Setting the size of a mp_image must be done with mp_image_set_size()
now. Do this to guarantee that the redundant fields (like chroma_width)
are updated consistently. Replacing the redundant fields by function
calls would probably be better, but there are too many uses of them,
and is a bit less convenient.
Most code actually called mp_image_setfmt(), which did this as well.
This commit just makes things a bit more explicit.
Warning: the video filter chain still sets up mp_images manually,
and vf_get_image() is not updated.
Diffstat (limited to 'video/out/vo_sdl.c')
| -rw-r--r-- | video/out/vo_sdl.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/video/out/vo_sdl.c b/video/out/vo_sdl.c index f5a17b84d3..d04a63e5a6 100644 --- a/video/out/vo_sdl.c +++ b/video/out/vo_sdl.c @@ -435,8 +435,7 @@ static int config(struct vo *vo, uint32_t width, uint32_t height, } mp_image_t *texmpi = &vc->texmpi; - texmpi->width = texmpi->w = width; - texmpi->height = texmpi->h = height; + mp_image_set_size(texmpi, width, height); mp_image_setfmt(texmpi, format); switch (texmpi->num_planes) { case 1: |