diff options
author | wm4 <wm4@nowhere> | 2016-12-02 15:26:45 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2016-12-02 15:26:45 +0100 |
commit | 09238a9bb536982e79b60a264b1d8a4fc0302975 (patch) | |
tree | 04591a027547a233e0e9bfecd05fa38f916000bb /video/out/opengl/utils.h | |
parent | ec74a79e1240eeda7b0bb195b484420052ea0ad8 (diff) | |
download | mpv-09238a9bb536982e79b60a264b1d8a4fc0302975.tar.bz2 mpv-09238a9bb536982e79b60a264b1d8a4fc0302975.tar.xz |
vo_opengl: don't rely on viewport to contain window dimensions
Apparently we don't always set the viewport to window dimensions
anymore, e.g. if nothing is actually rendered. This means the viewport
can contain old values.
The window screenshot code uses the viewport values to guess the default
framebuffer dimensions. With --force-window --idle --no-osc (which draws
nothing and issues a glClear() command only), taking a screenshot would
yield an image with the wrong size and possibly garbage in it. Fix this
by explicitly passing the currently known window dimensions. Abusing the
values stored in the viewport was questionable anyway.
Diffstat (limited to 'video/out/opengl/utils.h')
-rw-r--r-- | video/out/opengl/utils.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/video/out/opengl/utils.h b/video/out/opengl/utils.h index 33c2daa5be..f4e522cdf7 100644 --- a/video/out/opengl/utils.h +++ b/video/out/opengl/utils.h @@ -30,7 +30,7 @@ void gl_upload_tex(GL *gl, GLenum target, GLenum format, GLenum type, const void *dataptr, int stride, int x, int y, int w, int h); -mp_image_t *gl_read_window_contents(GL *gl); +mp_image_t *gl_read_window_contents(GL *gl, int w, int h); const char* mp_sampler_type(GLenum texture_target); |