February 2016 - MPlayer-dev-eng - lists.mplayerhq.hu

Relicensing MPlayer or parts of it to LGPL
by wm4 27 Sep '16

27 Sep '16

I'm maintainer of mpv, a MPlayer fork. One of the changes in mpv was turning the slave mode interface into a library (libmpv), which replaces the stdin/stdout parsing slave mode MPlayer uses. One problem with this is that it forces every client to be GPL. libmpv basically exhibits the slave mode interface as API, but since the application code actually links to it, and the player runs in the same address space, the GPL technically applies (in my interpretation). To avoid this, I'm thinking of relicensing mpv to LGPL 2.1 (with the "or later"). While many things have changed in mpv compared to MPlayer (and many parts have been rewritten or removed), much of its core is still based on MPlayer, and I need agreement from the original authors of the code. I'm thinking it would be beneficial for MPlayer as well to relicense parts to LGPL (or all of it, if enough contributors can be reached). Alternatively, I'd just ask for permission to relicense the affected source files in mpv. Would there be any welcoming or opposing to such a move from MPlayer developers?

8 20

[PATCH] Added support for arm64
by jpinto 24 May '16

24 May '16

Signed-off-by: Joao Pinto <jpinto(a)synopsys.com> Tested-by: Joao Pinto <jpinto(a)synopsys.com> --- configure | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/configure b/configure index 722b8a4..d17e61e 100755 --- a/configure +++ b/configure @@ -2493,6 +2493,11 @@ EOF arch='generic' ;; + aarch64) + arch='arm64' + iproc='arm64' + ;; + *) echo "The architecture of your CPU ($host_arch) is not supported by this configure script" echo "It seems nobody has ported MPlayer to your OS or CPU type yet." -- 1.8.1.5

3 3

[PATCH] configure: Avoid eh_frame bloat in release builds
by Lauri Kasanen 10 Mar '16

10 Mar '16

Hi, This patch reduces both mplayer and mencoder size by 1mb, or 6%. The eh_frame sections are mainly used for C++ exceptions and accurate debugging, neither of which is desired in a C program and a release build. - Lauri

3 13

MPlayer and raw video
by Mats Peterson 26 Feb '16

26 Feb '16

Since there for some reason hasn't been any response to my previous posting, here we go again: MPlayer naively assumes that 'rawvideo' raw video from libavcodec (whether it's contained in AVI, QuickTime or something else) contains 24-bit RGB triplets. It *can*, for pixel formats like rgb24 and bgr24, but it can also be less than 24 bits (palettized, grayscale or monochrome). Anything that's not compressed/encoded. What's more, MPlayer misinterprets a negative height in the BITMAPINFOHEADER for AVI files as being a very large unsigned value, rather than the indication of the video data being arranged top-to-bottom rather than bottom-to-top. The files below with raw video will play without video in MPlayer. QuickTime 1 bpp: https://drive.google.com/open?id=0B3_pEBoLs0faU05mTVJ3b3VZcmc QuickTime 2 bpp: https://drive.google.com/open?id=0B3_pEBoLs0faZUc4VEVwalhGSE0 QuickTime 4 bpp: https://drive.google.com/open?id=0B3_pEBoLs0facHJkWnhvYXdtbms QuickTime 8 bpp: https://drive.google.com/open?id=0B3_pEBoLs0fabUdRcTVxRl9yVWM AVI 1 bpp: https://drive.google.com/open?id=0B3_pEBoLs0faelN5RXpKLWtfTnM AVI 2 bpp: https://drive.google.com/open?id=0B3_pEBoLs0fabnQyNm5VV2NXNGM AVI 4 bpp: https://drive.google.com/open?id=0B3_pEBoLs0faTnppb3NjdFQzMmc AVI 8 bpp: https://drive.google.com/open?id=0B3_pEBoLs0faekxWUTdkVGNTTlk Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/

3 19

[PATCH] configure: Add support for OpenSSL
by Lauri Kasanen 22 Feb '16

22 Feb '16

Hi, This patch adds support for OpenSSL. Tested with a Youtube https stream. - Lauri

4 11

[PATCH v3 0/5] pvr:// support for hdpvr
by Reza Arbab 21 Feb '16

21 Feb '16

These changes enable pvr:// for the Hauppauge HD PVR model 1212. v3: Fix the "on first read" logic added in v2. v2: Revised using feedback from Reimar Döffinger. Thanks, Reimar! Reza Arbab (5): stream_pvr: refactor add_v4l2_ext_control stream_pvr: validate extended v4l2 controls before setting them stream_pvr: add AAC and AC-3 audio encoding control options stream_pvr: fix bogus error message on device poll timeout stream_pvr: allow initial latency when opening stream DOCS/man/cs/mplayer.1 | 2 +- DOCS/man/de/mplayer.1 | 2 +- DOCS/man/en/mplayer.1 | 2 +- DOCS/man/es/mplayer.1 | 2 +- DOCS/man/fr/mplayer.1 | 2 +- DOCS/man/hu/mplayer.1 | 2 +- DOCS/man/it/mplayer.1 | 2 +- DOCS/man/ru/mplayer.1 | 2 +- DOCS/man/zh_CN/mplayer.1 | 2 +- cfg-common.h | 2 +- stream/stream_pvr.c | 189 +++++++++++++++++++++++++++++++++++++++-------- 11 files changed, 168 insertions(+), 41 deletions(-) -- 2.5.0

3 12

[PATCH] Test the appropriate header for the system
by Ingo Brückl 21 Feb '16

21 Feb '16

This is an old patch having lain around here. I found it recently when cleaning up some directories. This kind of header check is already done for _struct_addrinfo, _getaddrinfo, _struct_sockaddr_storage and _struct_sockaddr_sa_len, so no need to do it differently and in a kind of brute force way for three other checks. Ingo

1 0

[PATCH] codecs.conf: add pixel formats for ffvp9.
by Nicolas George 21 Feb '16

21 Feb '16

The following pixel formats are present in the source too but have no MPlayer name: AV_PIX_FMT_YUV440P10 AV_PIX_FMT_YUV440P12 AV_PIX_FMT_GBRP10 AV_PIX_FMT_GBRP12 --- etc/codecs.conf | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) Tested (and necessary) with 444P, other formats taken from the source code (read_colorspace_details() in vp9.c). Also tested with GBR24P: it starts working instead of flooding with error messages but the colors are wrong; colors are wrong the same way with ffplay, so I guess it is a bug in lavc, I will investigate if I have time. Wrong colors are still better than a flood of error messages. Will commit shortly if nobody objects, and even faster if someone approves. diff --git a/etc/codecs.conf b/etc/codecs.conf index b18829a..d8010eb 100644 --- a/etc/codecs.conf +++ b/etc/codecs.conf @@ -3015,7 +3015,11 @@ videocodec ffvp9 fourcc VP90 driver ffmpeg dll "vp9" - out YV12 + out YV12,420P10,420P12 + out 422P,422P10,422P12 + out 440P + out 444P,444P10,444P12 + out GBR24P videocodec fflibvpx info "FFmpeg wrapper for libvpx/VP8" -- 2.7.0

4 4

[PATCH] Fix broken build by checking for used structure definition
by Erik Auerswald 20 Feb '16

20 Feb '16

Hi, the attached patch fixes a build error if the header linux/videodev2.h defines struct v4l2_ext_controls, but not struct v4l2_query_ext_ctrl. This is the case on Ubuntu 14.04.4 LTS, for example. Build error: ---8<------8<------8<------8<------8<------8<------8<------8<--- stream/stream_pvr.c: In function 'add_v4l2_ext_control': stream/stream_pvr.c:1099:10: error: variable 'qctrl' has initializer but incomplete type struct v4l2_query_ext_ctrl qctrl = { .id = id }; ^ stream/stream_pvr.c:1099:10: error: unknown field 'id' specified in initializer stream/stream_pvr.c:1099:10: warning: excess elements in struct initializer [enabled by default] stream/stream_pvr.c:1099:10: warning: (near initialization for 'qctrl') [enabled by default] stream/stream_pvr.c:1099:30: error: storage size of 'qctrl' isn't known struct v4l2_query_ext_ctrl qctrl = { .id = id }; ^ stream/stream_pvr.c:1102:27: error: 'VIDIOC_QUERY_EXT_CTRL' undeclared (first use in this function) if (ioctl (pvr->dev_fd, VIDIOC_QUERY_EXT_CTRL, &qctrl) < 0) ^ stream/stream_pvr.c:1102:27: note: each undeclared identifier is reported only once for each function it appears in stream/stream_pvr.c:1099:30: warning: unused variable 'qctrl' [-Wunused-variable] struct v4l2_query_ext_ctrl qctrl = { .id = id }; ^ make: *** [stream/stream_pvr.o] Error 1 make: *** Waiting for unfinished jobs.... ---8<------8<------8<------8<------8<------8<------8<------8<--- Thanks, Erik -- http://www.unix-ag.uni-kl.de/~auerswal/

2 1

[PATCH] fix libass fails with -subcp enca: and external subtitles try 2
by Basin Ilya 20 Feb '16

20 Feb '16

Hi. Here's a proposed patch for https://trac.mplayerhq.hu/ticket/2281 --- mplayer.c | 3 ++- sub/subreader.c | 8 ++++---- sub/subreader.h | 1 + 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/mplayer.c b/mplayer.c index aa5ca33..017e149 100644 --- a/mplayer.c +++ b/mplayer.c @@ -1085,11 +1085,12 @@ void add_subtitles(char *filename, float fps, int noerr) if (filename == NULL || mpctx->set_of_sub_size >= MAX_SUBTITLE_FILES) return; + enca_sub_cp = NULL; subd = sub_read_file(filename, fps); #ifdef CONFIG_ASS if (ass_enabled) #ifdef CONFIG_ICONV - asst = ass_read_stream(ass_library, filename, sub_cp); + asst = ass_read_stream(ass_library, filename, (enca_sub_cp ? enca_sub_cp : sub_cp)); #else asst = ass_read_stream(ass_library, filename, 0); #endif diff --git a/sub/subreader.c b/sub/subreader.c index ecc5ff5..510c43a 100644 --- a/sub/subreader.c +++ b/sub/subreader.c @@ -54,6 +54,7 @@ #include <iconv.h> #endif char *sub_cp=NULL; +char *enca_sub_cp=NULL; #ifdef CONFIG_FRIBIDI #include <fribidi/fribidi.h> char *fribidi_charset = NULL; ///character set that will be passed to FriBiDi @@ -1212,21 +1213,20 @@ void subcp_open (stream_t *st) char *tocp = "UTF-8"; if (sub_cp){ - const char *cp_tmp = sub_cp; #ifdef CONFIG_ENCA char enca_lang[3], enca_fallback[100]; if (sscanf(sub_cp, "enca:%2s:%99s", enca_lang, enca_fallback) == 2 || sscanf(sub_cp, "ENCA:%2s:%99s", enca_lang, enca_fallback) == 2) { if (st && st->flags & MP_STREAM_SEEK ) { - cp_tmp = guess_cp(st, enca_lang, enca_fallback); + enca_sub_cp = guess_cp(st, enca_lang, enca_fallback); } else { - cp_tmp = enca_fallback; + enca_sub_cp = enca_fallback; if (st) mp_msg(MSGT_SUBREADER,MSGL_WARN,"SUB: enca failed, stream must be seekable.\n"); } } #endif - if ((icdsc = iconv_open (tocp, cp_tmp)) != (iconv_t)(-1)){ + if ((icdsc = iconv_open (tocp, enca_sub_cp)) != (iconv_t)(-1)){ mp_msg(MSGT_SUBREADER,MSGL_V,"SUB: opened iconv descriptor.\n"); sub_utf8 = 2; } else diff --git a/sub/subreader.h b/sub/subreader.h index 0760081..554dbd7 100644 --- a/sub/subreader.h +++ b/sub/subreader.h @@ -33,6 +33,7 @@ extern int sub_no_text_pp; // disable text post-processing extern int sub_match_fuzziness; extern int sub_format; extern char *sub_cp; +extern char *enca_sub_cp; // subtitle formats #define SUB_INVALID -1 -- 2.7.0

2 3