Re: [FFmpeg-devel] [PATCH v2 1/4] ffprobe: add -show_pixel_formats option

2014-09-22 Thread Tobias Rapp
On 15.09.2014 17:00, Tobias Rapp wrote: Have updated my work according to the suggestions of Stefano and split into four patches for easier review. Any feedback/comments are welcome. Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] ffprobe shows errors on ts file

2014-09-22 Thread airwolf
On 09/20/2014 02:36 AM, Michael Niedermayer wrote: Hi, following file shows a bunch of errors in ffprobe: https://www.dropbox.com/s/04djuv4zn0l4lkx/espn.ts?dl=0 I'm not an expert in decoding that's why I try to ask here. I converted the file to mp4 and used the mp4toannexb filter, then I used

[FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Benoit Fouet
Considering the palette is located at the end of extradata may be flawed when the extradata contains the palette followed by the BottomUp field. When the BottomUp field is present, exclude it from the palette. Fixes ticket #1304 --- libavformat/avidec.c | 4 1 file changed, 4 insertions(+)

Re: [FFmpeg-devel] [PATCH] avformat/img2: remove useless 'pix' duplicated entry.

2014-09-22 Thread Benoit Fouet
ping? - Mail original - The second entry will never be selected, so let's save some bytes in the library. --- libavformat/img2.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/img2.c b/libavformat/img2.c index 8002054..d6f1244 100644 --- a/libavformat/img2.c

Re: [FFmpeg-devel] [PATCH] avformat/img2: remove useless 'pix' duplicated entry.

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 10:06:21AM +0200, Benoit Fouet wrote: ping? - Mail original - The second entry will never be selected, so let's save some bytes in the library. --- libavformat/img2.c | 1 - 1 file changed, 1 deletion(-) applied thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 11:28:31AM +0200, Benoit Fouet wrote: Hi, - Mail original - On Mon, Sep 22, 2014 at 09:57:37AM +0200, Benoit Fouet wrote: Considering the palette is located at the end of extradata may be flawed when the extradata contains the palette followed by the

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Benoit Fouet
Hi, - Mail original - On Mon, Sep 22, 2014 at 11:28:31AM +0200, Benoit Fouet wrote: Hi, - Mail original - On Mon, Sep 22, 2014 at 09:57:37AM +0200, Benoit Fouet wrote: Considering the palette is located at the end of extradata may be flawed when the

[FFmpeg-devel] [PATCH] Add missing entry for maintainer of rtpenc_hevc.*

2014-09-22 Thread Thomas Volkert
From: Thomas Volkert tho...@homer-conferencing.com --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 545b71e..90a4013 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -463,7 +463,7 @@ Muxers/Demuxers: rtmp*

Re: [FFmpeg-devel] [PATCH] Add missing entry for maintainer of rtpenc_hevc.*

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 01:07:10PM +0200, Thomas Volkert wrote: From: Thomas Volkert tho...@homer-conferencing.com --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB In fact,

Re: [FFmpeg-devel] [PATCH] avformat/avidec: ensure that palette does not contain the BottomUp info.

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 12:14:53PM +0200, Benoit Fouet wrote: Hi, - Mail original - On Mon, Sep 22, 2014 at 11:28:31AM +0200, Benoit Fouet wrote: Hi, - Mail original - On Mon, Sep 22, 2014 at 09:57:37AM +0200, Benoit Fouet wrote: Considering the palette is

Re: [FFmpeg-devel] ffprobe shows errors on ts file

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 09:37:53AM +0200, airwolf wrote: On 09/20/2014 02:36 AM, Michael Niedermayer wrote: Hi, following file shows a bunch of errors in ffprobe: https://www.dropbox.com/s/04djuv4zn0l4lkx/espn.ts?dl=0 I'm not an expert in decoding that's why I try to ask here. I

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Rainer Hochecker
Clément Bœsch u at pkh.me writes: +1 Applied We at XBMC got the first complaints about broken subtitles. Why was this removed? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread wm4
On Mon, 22 Sep 2014 14:53:12 + (UTC) Rainer Hochecker fernetme...@online.de wrote: Clément Bœsch u at pkh.me writes: +1 Applied We at XBMC got the first complaints about broken subtitles. Why was this removed? It never worked right in the first place, at least not

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Clément Bœsch
On Mon, Sep 22, 2014 at 02:53:12PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: +1 Applied We at XBMC got the first complaints about broken subtitles. Why was this removed? Hi, We just made the ASS packets from the Matroska output them verbatim, and

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Rainer Hochecker
Clément Bœsch u at pkh.me writes: Hi, We just made the ASS packets from the Matroska output them verbatim, and not hacked with timestamps included into the timestamps. The packets are now typically broadcastable to libass. Note that this change has been documented since 2.0;

Re: [FFmpeg-devel] [PATCH v3 1/1] configure: Refactor setting of feature test macro _XOPEN_SOURCE

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 05:13:24PM +0200, Michael Niedermayer wrote: On Sun, Sep 21, 2014 at 10:48:01PM +0200, Jörg Krause wrote: Some C libraries, eg glibc, uclibc, and musl, uses feature test macros to expose definitions conforming to the standards ISO C, POSIX and extensions. According

Re: [FFmpeg-devel] [PATCH v3 1/1] configure: Refactor setting of feature test macro _XOPEN_SOURCE

2014-09-22 Thread wm4
On Mon, 22 Sep 2014 17:13:24 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Sep 21, 2014 at 10:48:01PM +0200, Jörg Krause wrote: Some C libraries, eg glibc, uclibc, and musl, uses feature test macros to expose definitions conforming to the standards ISO C, POSIX and

Re: [FFmpeg-devel] [PATCH v3 1/1] configure: Refactor setting of feature test macro _XOPEN_SOURCE

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 06:19:42PM +0200, wm4 wrote: On Mon, 22 Sep 2014 17:13:24 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Sep 21, 2014 at 10:48:01PM +0200, Jörg Krause wrote: Some C libraries, eg glibc, uclibc, and musl, uses feature test macros to expose definitions

Re: [FFmpeg-devel] [PATCH] avformat: Support merging side data before the muxer

2014-09-22 Thread Nicolas George
Le primidi 1er vendémiaire, an CCXXIII, Michael Niedermayer a écrit : i dont know details about the application/lib using this, this patch was just the result of some investigation into why this other patch was needed: [FFmpeg-devel] [PATCH 1/2] mux: make sure that the AV_PKT_DATA_H263_MB_INFO

Re: [FFmpeg-devel] [PATCH] avformat: Support merging side data before the muxer

2014-09-22 Thread Thomas Volkert
Hello, On 22.09.2014 19:02, Nicolas George wrote: In general I can agree with that argument, but in this particular case I believe it does not apply for two reasons: First, the application will need some redesign anyway to make use of the merged-side-data hack. Definitely not - but this is

Re: [FFmpeg-devel] [PATCH] libavcodec/dvdsubdec: Add option forced_subs_only to only decode forced subtitle frames.

2014-09-22 Thread Nicholas Robbins
ping On Saturday, September 20, 2014 7:54 AM, Nicholas Robbins nickrobbins-at-yahoo@ffmpeg.org wrote: Signed-off-by: Nicholas Robbins nickrobb...@yahoo.com --- libavcodec/dvdsubdec.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/dvdsubdec.c

Re: [FFmpeg-devel] [PATCH v3 1/1] configure: Refactor setting of feature test macro _XOPEN_SOURCE

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 06:39:51PM +0200, wm4 wrote: On Mon, 22 Sep 2014 18:31:36 +0200 Michael Niedermayer michae...@gmx.at wrote: On Mon, Sep 22, 2014 at 06:19:42PM +0200, wm4 wrote: On Mon, 22 Sep 2014 17:13:24 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Sep

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Rainer Hochecker
Clément Bœsch u at pkh.me writes: On Mon, Sep 22, 2014 at 03:12:02PM +, Rainer Hochecker wrote: [...] Hi, Thank you very much for this explanation and sorry for not having read release notes properly. subs are not my territory but the maintainer is busy with real life so I will

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Clément Bœsch
On Mon, Sep 22, 2014 at 06:55:54PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: On Mon, Sep 22, 2014 at 03:12:02PM +, Rainer Hochecker wrote: [...] Hi, Thank you very much for this explanation and sorry for not having read release notes properly. subs

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread wm4
On Mon, 22 Sep 2014 21:05:19 +0200 Clément Bœsch u...@pkh.me wrote: On Mon, Sep 22, 2014 at 06:55:54PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: On Mon, Sep 22, 2014 at 03:12:02PM +, Rainer Hochecker wrote: [...] Hi, Thank you very much for

Re: [FFmpeg-devel] [PATCH] libavcodec/dvdsubdec: Add option forced_subs_only to only decode forced subtitle frames.

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 11:14:25AM -0700, Nicholas Robbins wrote: ping applied also, please add documentation for the option thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is dangerous to be right in matters on which the established authorities

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Clément Bœsch
On Mon, Sep 22, 2014 at 09:13:18PM +0200, wm4 wrote: On Mon, 22 Sep 2014 21:05:19 +0200 Clément Bœsch u...@pkh.me wrote: On Mon, Sep 22, 2014 at 06:55:54PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: On Mon, Sep 22, 2014 at 03:12:02PM +, Rainer

[FFmpeg-devel] [PATCH] doc/decoders: adding documentation for lavc/dvdsubdec.c option forced_subs_only

2014-09-22 Thread Nicholas Robbins
Signed-off-by: Nicholas Robbins nickrobb...@yahoo.com --- doc/decoders.texi | 5 + 1 file changed, 5 insertions(+) diff --git a/doc/decoders.texi b/doc/decoders.texi index 6910dc1..56ce928 100644 --- a/doc/decoders.texi +++ b/doc/decoders.texi @@ -190,6 +190,11 @@ The format for this option

Re: [FFmpeg-devel] [PATCH] doc/decoders: adding documentation for lavc/dvdsubdec.c option forced_subs_only

2014-09-22 Thread Nicholas Robbins
On Monday, September 22, 2014 3:28 PM, Nicholas Robbins nickrobbins-at-yahoo@ffmpeg.org wrote: +Only decoded subtitle entries marked as forced. Some titles have forced Should be 'decode' not 'decoded'. Resent patch -Nick ___ ffmpeg-devel

[FFmpeg-devel] [PATCH 2/2] configure: Refactor setting of feature test macro _XOPEN_SOURCE

2014-09-22 Thread Michael Niedermayer
From: Jörg Krause jkra...@posteo.de Some C libraries, eg glibc, uclibc, and musl, uses feature test macros to expose definitions conforming to the standards ISO C, POSIX and extensions. According to which feature test macros are defined by the user or the compiler, a header file, eg features.h,

Re: [FFmpeg-devel] ffplay -fs fix (ticket #3964)

2014-09-22 Thread Marton Balint
On Fri, 19 Sep 2014, Jeff Dwork wrote: On MS Windows, 'ffplay.exe -fs' does not start fullscreen. Behavior on linux is correct. On Windows, the initial placement of the SDL screen triggers the SDL_VIDEORESIZE event. This executes the code in the event_loop which calls SDL_SetVideoMode with

Re: [FFmpeg-devel] [PATCH]Allow using libopenjpeg 2.x

2014-09-22 Thread Carl Eugen Hoyos
On Wednesday 17 September 2014 09:16:06 am Reimar Döffinger wrote: On 16.09.2014, at 22:35, Carl Eugen Hoyos ceho...@ag.or.at wrote: Attached patch allows using libopenjpeg 2.x (tested with 2.1) here. Doesn't it make sense to prefer the newest version, i.e. we should check for this first and

Re: [FFmpeg-devel] [PATCH] doc/decoders: adding documentation for lavc/dvdsubdec.c option forced_subs_only

2014-09-22 Thread Michael Niedermayer
On Mon, Sep 22, 2014 at 03:31:35PM -0400, Nicholas Robbins wrote: Signed-off-by: Nicholas Robbins nickrobb...@yahoo.com --- doc/decoders.texi | 5 + 1 file changed, 5 insertions(+) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is

[FFmpeg-devel] [PATCH] avcodec/webp: fix default palette color and optimize for large image

2014-09-22 Thread Pascal Massimino
Hi, as discussed on the other thread... skal 0001-avcodec-webp-fix-default-palette-color-0xff00-0x.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 2/2] configure: Check that localtime_r() is in a header and if not retry with XOPEN/POSIX_SOURCE

2014-09-22 Thread Michael Niedermayer
This fixes the localtime_r() check, which was producing a false positive with musl and also fixes build with musl Signed-off-by: Michael Niedermayer michae...@gmx.at --- configure |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index

Re: [FFmpeg-devel] [PATCH] avcodec/webp: fix default palette color and optimize for large image

2014-09-22 Thread Pascal Massimino
On Mon, Sep 22, 2014 at 3:07 PM, Pascal Massimino pascal.massim...@gmail.com wrote: Hi, as discussed on the other thread... and now, a version with the correct indentation, sorry. skal 0002-avcodec-webp-fix-default-palette-color-0xff00-0x.patch Description: Binary data

Re: [FFmpeg-devel] [PATCH 1/2] configure: support --disable-sdl

2014-09-22 Thread Reimar Döffinger
On Tue, Sep 23, 2014 at 12:42:53AM +0200, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- configure | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 12b2da5..f674a06

Re: [FFmpeg-devel] [PATCH 1/2] configure: support --disable-sdl

2014-09-22 Thread Michael Niedermayer
On Tue, Sep 23, 2014 at 01:01:26AM +0200, Reimar Döffinger wrote: On Tue, Sep 23, 2014 at 12:42:53AM +0200, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- configure | 33 ++--- 1 file changed, 18 insertions(+), 15

Re: [FFmpeg-devel] [PATCH] ffmpeg: replace missing PTS by DTS

2014-09-22 Thread Clément Bœsch
On Tue, Sep 23, 2014 at 03:21:30AM +0200, Michael Niedermayer wrote: This fixes the case where muxing fails hard in case of stream copying damaged input Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffmpeg.c |8 1 file changed, 8 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH] ffmpeg: replace missing PTS by DTS

2014-09-22 Thread Clément Bœsch
On Tue, Sep 23, 2014 at 07:45:56AM +0200, Clément Bœsch wrote: On Tue, Sep 23, 2014 at 03:21:30AM +0200, Michael Niedermayer wrote: This fixes the case where muxing fails hard in case of stream copying damaged input Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffmpeg.c |

Re: [FFmpeg-devel] ffplay -fs fix (ticket #3964)

2014-09-22 Thread Jeff Dwork
Your patch works. Thanks, Jeff At 01:19 PM 9/22/2014, you wrote: On Fri, 19 Sep 2014, Jeff Dwork wrote: On MS Windows, 'ffplay.exe -fs' does not start fullscreen. Behavior on linux is correct. On Windows, the initial placement of the SDL screen triggers the SDL_VIDEORESIZE event. This