From cf61928eecd87ba29324e501a3e4a0efde25d874 Mon Sep 17 00:00:00 2001 From: wm4 Date: Wed, 3 Oct 2012 16:50:13 +0200 Subject: vo_gl, options: remove doublebuffering option (--double) Useless. It complicated the code and caused flicker, and was useless otherwise. The manpage describes this option as "should not normally be used". One possibly useful effect from the point of view of the user was that vsync was disabled. You can do this with the --vsync option, or by changing X/driver settings directly. --- DOCS/man/en/options.rst | 4 ---- 1 file changed, 4 deletions(-) (limited to 'DOCS/man') diff --git a/DOCS/man/en/options.rst b/DOCS/man/en/options.rst index c52c8b307e..bf297018a9 100644 --- a/DOCS/man/en/options.rst +++ b/DOCS/man/en/options.rst @@ -489,10 +489,6 @@ ``--display=xtest.localdomain:0`` ---double, --no-double - Double buffering. The option to disable this exists mostly for debugging - purposes and should not normally be used. - --doubleclick-time Time in milliseconds to recognize two consecutive button presses as a double-click (default: 300). -- cgit v1.2.3 From 8f8f6e6d9d27eef37640b363214a775647a0f2f4 Mon Sep 17 00:00:00 2001 From: wm4 Date: Wed, 3 Oct 2012 17:00:37 +0200 Subject: sub: remove logic for disabling hinting on scaled EOSD This was an extremely obscure setting, as it was used only with vo_gl if its scaled-osd suboption was used. If you really want this, you can set the desired ass-hinting value directly, and there will be literally no loss in functionality. Note that this didn't actually test whether the EOSD was scaled. Basically, it only checked whether vo_gl had the scaled-osd suboption set. --- DOCS/man/en/options.rst | 2 -- 1 file changed, 2 deletions(-) (limited to 'DOCS/man') diff --git a/DOCS/man/en/options.rst b/DOCS/man/en/options.rst index bf297018a9..0b623859e5 100644 --- a/DOCS/man/en/options.rst +++ b/DOCS/man/en/options.rst @@ -163,8 +163,6 @@ :1: FreeType autohinter, light mode :2: FreeType autohinter, normal mode :3: font native hinter - :0-3 + 4: The same, but hinting will only be performed if the OSD is - rendered at screen resolution and will therefore not be scaled. The default value is 0 (no hinting). -- cgit v1.2.3 From fd5c4a19849b768986a0e652bb2f398c922f42dd Mon Sep 17 00:00:00 2001 From: wm4 Date: Sun, 7 Oct 2012 22:02:03 +0200 Subject: Remove things related to old OSD To ease changing all the VOs to the new OSD rendering, fallbacks, conversions, support code etc. was left all over the code. Now that all VOs have been changed, all that code is inactive. Remove it. Strip down spudec.c. We don't need the old grayscale and scaling stuff anymore. (Not removing spudec itself yet - I'm not confident that the libavcodec DVD sub decoder is sufficient, and it would also require some hacks to get DVD palette and resolution information from libdvdread to libavcodec.) The option --spuaa, --spualign, --spugauss were used with the old sub scaling code, and don't do anything anymore. --- DOCS/man/en/options.rst | 24 ------------------------ 1 file changed, 24 deletions(-) (limited to 'DOCS/man') diff --git a/DOCS/man/en/options.rst b/DOCS/man/en/options.rst index ca245f6d32..3be707bfad 100644 --- a/DOCS/man/en/options.rst +++ b/DOCS/man/en/options.rst @@ -1781,30 +1781,6 @@ --speed=<0.01-100> Slow down or speed up playback by the factor given as parameter. ---spuaa= - Antialiasing/scaling mode for DVD/VOBsub. A value of 16 may be added to - in order to force scaling even when original and scaled frame size - already match. This can be employed to e.g. smooth subtitles with gaussian - blur. Available modes are: - - :0: none (fastest, very ugly) - :1: approximate (broken?) - :2: full (slow) - :3: bilinear (default, fast and not too bad) - :4: uses swscaler gaussian blur (looks very good) - ---spualign=<-1-2> - Specify how SPU (DVD/VOBsub) subtitles should be aligned. - - :-1: Original position - :0: Align at top (original behavior, default). - :1: Align at center. - :2: Align at bottom. - ---spugauss=<0.0-3.0> - Variance parameter of gaussian used by ``--spuaa=4``. Higher means more - blur (default: 1.0). - --srate= Select the output sample rate to be used (of course sound cards have limits on this). If the sample frequency selected is different from that -- cgit v1.2.3 From 4d11f32162b08e3b48ae382e2ed0a151035f8aea Mon Sep 17 00:00:00 2001 From: wm4 Date: Fri, 19 Oct 2012 19:25:18 +0200 Subject: VO, sub: refactor Remove VFCTRL_DRAW_OSD, VFCAP_EOSD_FILTER, VFCAP_EOSD_RGBA, VFCAP_EOSD, VOCTRL_DRAW_EOSD, VOCTRL_GET_EOSD_RES, VOCTRL_QUERY_EOSD_FORMAT. Remove draw_osd_with_eosd(), which rendered the OSD by calling VOCTRL_DRAW_EOSD. Change VOs to call osd_draw() directly, which takes a callback as argument. (This basically works like the old OSD API, except multiple OSD bitmap formats are supported and caching is possible.) Remove all mentions of "eosd". It's simply "osd" now. Make OSD size per-OSD-object, as they can be different when using vf_sub. Include display_par/video_par in resolution change detection. Fix the issue with margin borders in vo_corevideo. --- DOCS/man/en/vf.rst | 6 +++--- DOCS/man/en/vo.rst | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'DOCS/man') diff --git a/DOCS/man/en/vf.rst b/DOCS/man/en/vf.rst index d46bd5bc06..b5a672e236 100644 --- a/DOCS/man/en/vf.rst +++ b/DOCS/man/en/vf.rst @@ -664,14 +664,14 @@ screenshot ``Taking screenshots`` section for details. ass - Moves SSA/ASS subtitle rendering to an arbitrary point in the filter + Moves subtitle rendering to an arbitrary point in the filter chain, or force subtitle rendering in the video filter as opposed to using - video output EOSD support. See the ``--ass`` option. + video output OSD support. *EXAMPLE*: ``--vf=ass,eq`` - Moves SSA/ASS rendering before the eq filter. This will put both + Moves sub rendering before the eq filter. This will put both subtitle colors and video under the influence of the video equalizer settings. diff --git a/DOCS/man/en/vo.rst b/DOCS/man/en/vo.rst index 6248073944..09d12e2994 100644 --- a/DOCS/man/en/vo.rst +++ b/DOCS/man/en/vo.rst @@ -171,8 +171,8 @@ direct3d_shaders (Windows only) Never render YUV video with more than 8 bits per component. (Using this flag will force software conversion to 8 bit.) - disable-eosd - Disable EOSD rendering for subtitles. + disable-osd + Disable OSD rendering for subtitles. (Using this flag might force the insertion of the 'ass' video filter, which will render the subtitles in software.) -- cgit v1.2.3 From 11ed093182f29c796ba6fd08f6c5c80e8fdb9349 Mon Sep 17 00:00:00 2001 From: wm4 Date: Sat, 20 Oct 2012 23:32:45 +0200 Subject: screenshot: allow taking screenshots with subtitles This adds a new screenshot mode "subtitles", which basically takes the video frame as decoded, and renders subtitles into it. This may fail for some pixel formats, because libswscale sucks. If this becomes ever a real problem, the code could be changed to convert the image to RGBA first (or whatever the image writer wants), and then render the subtitles into it. This would avoid the additional image copy needed with vo_xv too. But for now, it seems better to go with the current method in the common case: vo_opengl creates an image copy anyway, and drawing bitmaps to yv12 is better, as no color space conversion is involved in draw_bmp.c's up/downsampling conversion. --- DOCS/man/en/input.rst | 3 +++ 1 file changed, 3 insertions(+) (limited to 'DOCS/man') diff --git a/DOCS/man/en/input.rst b/DOCS/man/en/input.rst index 74c4929684..63593c53ca 100644 --- a/DOCS/man/en/input.rst +++ b/DOCS/man/en/input.rst @@ -104,6 +104,9 @@ screenshot [single|each-frame] [video|window] Save the video image, in its original resolution. Typically without OSD or subtitles, but the exact behavior depends on the selected video output. + + Like ``video``, but add subtitles. Some video outputs may still include + the OSD in the output under certain circumstances. Save the contents of the mplayer window. Typically scaled, with OSD and subtitles. The exact behavior depends on the selected video output, and -- cgit v1.2.3 From 48ce4ab7f9259977f426d52b51601a54dbe15465 Mon Sep 17 00:00:00 2001 From: wm4 Date: Sat, 20 Oct 2012 23:58:02 +0200 Subject: screenshot: change "screenshot" input command "screenshot" now maps to "screenshot subtitles" by default, instead of "screenshot video". Swap the argument order: the more useful argument should come first. Remove the compatibility aliases for numeric choices (e.g. "screenshot 1 0" won't work anymore). --- DOCS/man/en/changes.rst | 3 ++- DOCS/man/en/input.rst | 30 +++++++++++++++--------------- DOCS/man/en/mpv.rst | 5 +++-- 3 files changed, 20 insertions(+), 18 deletions(-) (limited to 'DOCS/man') diff --git a/DOCS/man/en/changes.rst b/DOCS/man/en/changes.rst index 402bdbd336..725385a2b1 100644 --- a/DOCS/man/en/changes.rst +++ b/DOCS/man/en/changes.rst @@ -52,7 +52,8 @@ General changes for mplayer2 to mpv * Improved OpenGL output (``vo_opengl``) * Make ``--softvol`` default (**mpv** is not a mixer control panel) * Improved support for .cue files -* Screenshot improvements (can save screenshots as JPG, configurable filenames) +* Screenshot improvements (can save screenshots as JPG, configurable filenames, + support not taking screenshots with or without subtitles) * Removal of teletext support * Replace image VOs (``vo_jpeg`` etc.) with ``vo_image`` * Remove ``vo_gif89a``, ``vo_md5sum``, ``vo_yuv4mpeg`` (the plan is to merge diff --git a/DOCS/man/en/input.rst b/DOCS/man/en/input.rst index 63593c53ca..18e5b824e1 100644 --- a/DOCS/man/en/input.rst +++ b/DOCS/man/en/input.rst @@ -87,31 +87,31 @@ cycle [up|down] speed_mult Multiply the ``speed`` property by the given value. -screenshot [single|each-frame] [video|window] +screenshot [subtitles|video|window] [single|each-frame] Take a screenshot. First argument: + (default) + Save the video image, in its original resolution, and with subtitles. + Some video outputs may still include the OSD in the output under certain + circumstances. +