Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_pixel_formats option (was: add -show_pixel_descriptions option)

2014-09-11 Thread Tobias Rapp
Have now added documentation and cleaned up some unused variables with the new patch. Also on a second thought the option names show_pixel_format_descriptions or show_pixel_formats seem less confusing to me than show_pixel_descriptions so I did choose the shorter show_pixel_formats. What I

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_pixel_formats option (was: add -show_pixel_descriptions option)

2014-09-11 Thread Stefano Sabatini
On date Thursday 2014-09-11 09:54:42 +0200, Tobias Rapp encoded: Have now added documentation and cleaned up some unused variables with the new patch. Also on a second thought the option names show_pixel_format_descriptions or show_pixel_formats seem less confusing to me than

Re: [FFmpeg-devel] [PATCH] lavf/segment: add segment_format_options option

2014-09-11 Thread Stefano Sabatini
On date Wednesday 2014-09-10 17:57:04 +0800, Steven Liu encoded: What about push this option into format hls? Please open a ticket on trac so I won't forget it (or alternatively provide a patch). -- FFmpeg = Fiendish Furious Mournful Perennial Extroverse Gospel

Re: [FFmpeg-devel] [PATCH] avformat/rmdec: very basic MLTI support

2014-09-11 Thread Michael Niedermayer
On Tue, Sep 09, 2014 at 01:09:49AM +0200, Michael Niedermayer wrote: Fixes Ticket2152 Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/rmdec.c | 15 +++ 1 file changed, 15 insertions(+) applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] lavf/segment: add segment_format_options option

2014-09-11 Thread Steven Liu
Okay, I'vemake a patch for the hls with this option :-) 2014-09-11 18:13 GMT+08:00 Stefano Sabatini stefa...@gmail.com: On date Wednesday 2014-09-10 17:57:04 +0800, Steven Liu encoded: What about push this option into format hls? Please open a ticket on trac so I won't forget it (or

Re: [FFmpeg-devel] [PATCH] lavf/hlsenc: add hls_ts_options option

2014-09-11 Thread Stefano Sabatini
On date Thursday 2014-09-11 19:52:07 +0800, Steven Liu encoded: Signed-off-by: Steven Liu qi@chinacache.com Signed-off-by: Stefano Sabatini stefa...@gmail.com Probably you should remove this, and mention the equivalent commit in my patch (referencing the hash is fine). ---

[FFmpeg-devel] [PATCH 2/4] avformat/img2dec: reduce bmppipe probe score

2014-09-11 Thread Michael Niedermayer
bmp pipe needs the bmp parser which is not bug free and should thus not be favored over the bmp image2 demuxer Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/img2dec.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/img2dec.c

[FFmpeg-devel] [PATCH 1/4] avformat/utils: return EINVAL from seek_frame_generic() when the requested target appears to be outside the file duration

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/utils.c |4 +-- tests/ref/seek/acodec-adpcm-swf |8 ++--- tests/ref/seek/acodec-adpcm-swf-trellis |8 ++--- tests/ref/seek/acodec-mp2|8 ++--- tests/ref/seek/lavf-avi

[FFmpeg-devel] [PATCH 4/4] avformat/img2dec: Fail probing when no data is yet available and the filename contains no number/glob patterns either.

2014-09-11 Thread Michael Niedermayer
Fixes Ticket3901 Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/img2dec.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index 9f48c5d..a21429f 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -162,6

Re: [FFmpeg-devel] [PATCH] img2dec: make probing score zero for known file extensions.

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 03:48:21PM +0200, Benoit Fouet wrote: When a known extension has its own probing function, make img_read_probe return 0, so that the probing functions are actually called instead of relying only on the file extensions. Fixes issue #3901 --- libavformat/img2dec.c |

Re: [FFmpeg-devel] [PATCH 1/1] configure: fix check_cmd in check_host_cpp

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 02:48:15PM +0200, Benoit Fouet wrote: Hi, - Mail original - Use correct cpp and c flags variables for the host libc. Signed-off-by: Jörg Krause jkra...@posteo.de --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 7:18 AM, Grace Ryan rongyan...@gmail.com wrote: Hi, I present this patch, which is to fix the bug for the calculation of 'ff_vector_fmul_add_altivec' when run '/libavutil/float_altivec-test' on POWER8 little endian. The fate test result can be found on

[FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-11 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 4fa1174..44273cb 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -80,7 +80,9 @@ static const int

[FFmpeg-devel] [PATCH 2/3] avfilter/ass: better log level mapping

2014-09-11 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 44273cb..c9c21a7 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -68,14 +68,14 @@ typedef

Re: [FFmpeg-devel] suggested patch: avfilter/vf_subtitles: add support for subtitles font scaling

2014-09-11 Thread Clément Bœsch
On Thu, Sep 11, 2014 at 09:21:03PM +0200, wm4 wrote: On Thu, 11 Sep 2014 03:58:07 +0400 Sergey serg...@mail.ru wrote: Hello. Recently I used ffmpeg to embed subtitles, and I needed to scale them. I thought original_size option scales subtitles, but it does not. So I wrote a short

[FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Lou Logan
Replace with FFmpeg git-howto. Signed-off-by: Lou Logan l...@lrcd.com --- src/contact | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/contact b/src/contact index 00cb44f..96f7378 100644 --- a/src/contact +++ b/src/contact @@ -26,11 +26,8 @@ text/plain or

[FFmpeg-devel] FreeSwitch Video decoding (consulting opportunity)

2014-09-11 Thread Luciano Panaro
Hi everyone, In the company I work at we are trying to build a decoder for a video format generated by Freeswitch [1], a SIP platform through which we are recording a+v. We are using a module from FS called mod_fsv (details below). We are able to decode audio successfully but video is not being

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
Lou Logan lou at lrcd.com writes: - a href=http://www.mplayerhq.hu/DOCS/tech/patches.txt; MPlayer patch guidelines/a I understand that this is not ideal but ... + a href=git-howto.htmlUsing git to develop FFmpeg/a ... while this contains a lot of useful information it is missing (nearly)

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Carl Eugen Hoyos
Timothy Gu timothygu99 at gmail.com writes: +t1 = vec_perm(d, edges, vcprm(s0,s1,s2,s3)); +t0 = vec_perm(edges, d, vcprm(s0,s1,s2,s3)); ...you are using the old variable names, which I am pretty sure won't compile. I cannot explain why but it compiles and passes fate

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 1:57 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Lou Logan lou at lrcd.com writes: - a href=http://www.mplayerhq.hu/DOCS/tech/patches.txt; MPlayer patch guidelines/a I understand that this is not ideal but ... + a href=git-howto.htmlUsing git to develop FFmpeg/a

Re: [FFmpeg-devel] Fix bug for POWER8 LE for the test '/libavutil/float_altivec-test'

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 1:59 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Timothy Gu timothygu99 at gmail.com writes: +t1 = vec_perm(d, edges, vcprm(s0,s1,s2,s3)); +t0 = vec_perm(edges, d, vcprm(s0,s1,s2,s3)); ...you are using the old variable names, which I am pretty sure

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread James Darnley
On 2014-09-11 23:00, Timothy Gu wrote: On Thu, Sep 11, 2014 at 1:57 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Lou Logan lou at lrcd.com writes: - a href=http://www.mplayerhq.hu/DOCS/tech/patches.txt; MPlayer patch guidelines/a I understand that this is not ideal but ... + a

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Lou Logan
On Thu, 11 Sep 2014 20:57:27 + (UTC), Carl Eugen Hoyos wrote: Lou Logan lou at lrcd.com writes: - a href=http://www.mplayerhq.hu/DOCS/tech/patches.txt; MPlayer patch guidelines/a I understand that this is not ideal but ... + a href=git-howto.htmlUsing git to develop FFmpeg/a

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 11:53:57PM +0200, Michael Niedermayer wrote: On Thu, Sep 11, 2014 at 09:13:29PM +0200, Clément Bœsch wrote: [...] btw, in [2] = AV_LOG_WARNING, /* MSGL_WARN */ [3] = AV_LOG_WARNING, /* undefined */ [4] = AV_LOG_INFO, /* MSGL_INFO */ [5] =

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread James Darnley
On 2014-09-11 23:10, James Darnley wrote: I support the change and I am willing to look at adding and adapting some relevant points from the mplayer text. Point 0, diffs please, is covered by our request for git format-patch in the patch submission checklist[1] point 2 and the git how-to

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
James Darnley james.darnley at gmail.com writes: Point 2, unified diffs, is covered by out request for patches. Are we doing to allow git diff dumps? While I always send unified diffs, some people (still) send other diffs, and one regular contributor sends diffs in his own format, so this

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Carl Eugen Hoyos
Lou Logan lou at lrcd.com writes: Anyway, we also have the quite verbose http://ffmpeg.org/developer.html But you did not replace the MPlayer link by this link but by one that explains git usage which I consider important but different. Carl Eugen

[FFmpeg-devel] [PATCH] x86/motion_est: port mmxext and sse2 sad functions to yasm

2014-09-11 Thread James Almer
Also add sse2 versions of sad16_x2, sad16_y2 and sad16_xy2, and a missing c-pix_abs[0][0] initialization. Signed-off-by: James Almer jamr...@gmail.com --- Not benched. TODO: Port mmx. libavcodec/x86/me_cmp.asm| 229 +++ libavcodec/x86/me_cmp_init.c |

[FFmpeg-devel] [PATCH 0/6] mplayer pacthes text missing points

2014-09-11 Thread James Darnley
This adds some text covering points from that mplayer text which are missing in our docs. I welcome your comments, particularly on the language used. The first two patches are really just cosmetic changes but I noticed them while reading the file. James Darnley (6): docs: add missing code

[FFmpeg-devel] [PATCH 1/6] docs: add missing code formatting around commands

2014-09-11 Thread James Darnley
--- doc/developer.texi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index 1e1d3b8..e51e97a 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -328,7 +328,7 @@ area changed: Short 1 line description details describing

[FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread James Darnley
This addresses point 11 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 5 + 1 file changed, 5 insertions(+) diff --git

[FFmpeg-devel] [PATCH 6/6] docs: mention when and how one should be added as an author

2014-09-11 Thread James Darnley
This addresses point 13 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 5 + 1 file changed, 5 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH 1/6] docs: add missing code formatting around commands

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: --- doc/developer.texi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) I would use @command{}. [...] Timothy ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 2/6] docs: add example around the suggested commit message format

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: It gets rendered inline in HTML and becomes not very clear. --- doc/developer.texi | 3 +++ 1 file changed, 3 insertions(+) OK. [...] Timothy ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH 3/6] docs: mention the preferred character to use in options

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: This addresses point 12 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link

Re: [FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: This addresses point 11 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link

Re: [FFmpeg-devel] [PATCH 4/6] docs: mention the comment policy

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: This addresses point 6 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link

Re: [FFmpeg-devel] [PATCH 6/6] docs: mention when and how one should be added as an author

2014-09-11 Thread Timothy Gu
On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: This addresses point 13 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link

Re: [FFmpeg-devel] [PATCH]Add a closing CRLF to Webm-manifest files

2014-09-11 Thread Carl Eugen Hoyos
Carl Eugen Hoyos cehoyos at ag.or.at writes: Attached patch fixes fate-webm-dash-manifest on aix. This is the failure: $ gmake SAMPLES=fate-suite/ fate-webm-dash-manifest TESTwebm-dash-manifest diff: Missing newline at the end of file tests/data/fate/webm-dash-manifest. ---

[FFmpeg-devel] [PATCH v2] web/contact: replace MPlayer guidelines link

2014-09-11 Thread Lou Logan
Replace with FFmpeg Developer Documentation and mention git-howto. Signed-off-by: Lou Logan l...@lrcd.com --- src/contact | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/contact b/src/contact index 00cb44f..1043ec9 100644 --- a/src/contact +++ b/src/contact @@

[FFmpeg-devel] [PATCH 1/2] avformat/network: add union for avoiding strict aliassing violations with sockaddr*

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/network.h |8 1 file changed, 8 insertions(+) diff --git a/libavformat/network.h b/libavformat/network.h index d89a62d..d4e1a89 100644 --- a/libavformat/network.h +++ b/libavformat/network.h @@ -72,6 +72,14 @@ int

[FFmpeg-devel] [PATCH 2/2] avformat/rtpproto: fix strict aliasing violations with sockaddr

2014-09-11 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/rtpproto.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/libavformat/rtpproto.c b/libavformat/rtpproto.c index a8cbd97..62f77da 100644 --- a/libavformat/rtpproto.c +++ b/libavformat/rtpproto.c

Re: [FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread Michael Niedermayer
On Fri, Sep 12, 2014 at 01:31:29AM +0200, James Darnley wrote: This addresses point 11 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link ---

Re: [FFmpeg-devel] [PATCH 2/6] docs: add example around the suggested commit message format

2014-09-11 Thread Michael Niedermayer
On Thu, Sep 11, 2014 at 04:33:39PM -0700, Timothy Gu wrote: On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: It gets rendered inline in HTML and becomes not very clear. --- doc/developer.texi | 3 +++ 1 file changed, 3 insertions(+) OK. applied thanks

Re: [FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread wm4
On Fri, 12 Sep 2014 01:31:29 +0200 James Darnley james.darn...@gmail.com wrote: This addresses point 11 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link

[FFmpeg-devel] [PATCH] add hls_ts_option into hlsenc

2014-09-11 Thread Steven Liu
same as COMMITID 4f5493fe2380ad4aba67759baa7d7d4437f2e776 --- doc/muxers.texi | 5 + libavformat/hlsenc.c | 24 +++- 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 57e81f4..cc62705dc 100644 --- a/doc/muxers.texi