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.

1 files changed, 2 insertions, 5 deletions

diff --git a/video/out/vo_opengl.c b/video/out/vo_opengl.c
index 759cb96ba7..fc94a86eac 100644
--- a/video/out/vo_opengl.c
+++ b/video/out/vo_opengl.c
@@ -1350,11 +1350,8 @@ static mp_image_t *get_screenshot(struct gl_priv *p)
                       image->planes[n], image->stride[n]);
     }
     gl->ActiveTexture(GL_TEXTURE0);
-
-    image->w = p->image_width;
-    image->h = p->image_height;
-    image->display_w = p->vo->aspdat.prew;
-    image->display_h = p->vo->aspdat.preh;
+    mp_image_set_size(image, p->image_width, p->image_height);
+    mp_image_set_display_size(image, p->vo->aspdat.prew, p->vo->aspdat.preh);
 
     mp_image_set_colorspace_details(image, &p->colorspace);