diff options
author | wm4 <wm4@nowhere> | 2018-02-07 20:18:36 +0100 |
---|---|---|
committer | Kevin Mitchell <kevmitch@gmail.com> | 2018-02-11 17:45:51 -0800 |
commit | 9f595f3a80eeaa0bfbda5702031f054f684f6123 (patch) | |
tree | 0667e6e7ab4c64dcfd28aa23e10690e1776a6f03 /video/out/vo_gpu.c | |
parent | 7b1e73139f73f446a472782e1465040c0e6b16dd (diff) | |
download | mpv-9f595f3a80eeaa0bfbda5702031f054f684f6123.tar.bz2 mpv-9f595f3a80eeaa0bfbda5702031f054f684f6123.tar.xz |
vo_gpu: make screenshots use the GL renderer
Using the GL renderer for color conversion will make sure screenshots
will use the same conversion as normal video rendering. It can do this
for all types of screenshots.
The logic when to write 16 bit PNGs changes. To approximate the old
behavior, we decide by looking whether the source video format has more
than 8 bits per component. We apply this logic even for window
screenshots. Also, 16 bit PNGs now always include an unused alpha
channel. The reason is that FFmpeg has RGB48 and RGBA64 formats, but no
RGB064. RGB48 is 3 bytes and usually not supported by GPUs for
rendering, so we have to use RGBA64, which forces an alpha channel.
Will break for users who use --target-trc and similar options.
I considered creating a new gl_video context, but it could double GPU
memory use, so I didn't.
This uses FBOs instead of glGetTexImage(), because that increases the
chance it could work on GLES (e.g. ANGLE). Untested. No support for the
Vulkan and D3D11 backends yet.
Fixes #5498. Also fixes #5240, because the code for reading back is not
used with the new code path.
Diffstat (limited to 'video/out/vo_gpu.c')
-rw-r--r-- | video/out/vo_gpu.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/video/out/vo_gpu.c b/video/out/vo_gpu.c index d3b27ca5bc..70c874fc54 100644 --- a/video/out/vo_gpu.c +++ b/video/out/vo_gpu.c @@ -188,6 +188,20 @@ static int control(struct vo *vo, uint32_t request, void *data) *(struct mp_image **)data = screen; return true; } + case VOCTRL_SCREENSHOT: { + struct vo_frame *frame = vo_get_current_vo_frame(vo); + if (frame) { + // Disable interpolation and such. + frame->redraw = true; + frame->repeat = false; + frame->still = true; + frame->pts = 0; + frame->duration = -1; + gl_video_screenshot(p->renderer, frame, data); + } + talloc_free(frame); + return true; + } case VOCTRL_LOAD_HWDEC_API: request_hwdec_api(vo); return true; |