Re: [FFmpeg-devel] [PATCH 2/4] avcodec/samidec: use ff_htmlmarkup_to_ass()

2015-08-27 Thread Yayoi Ukai
Thank you for the suggestion.. It took me a while to get it but I think it looks much better now. Please let me know what you think! Cheers, On Sat, Aug 15, 2015 at 11:24 AM, Clément Bœsch wrote: > On Sat, Aug 08, 2015 at 09:24:03PM -0700, Yayoi Ukai wrote: >> On Sat, Aug 8, 2015 at 8:23 AM, C

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/strdec: factor out HTML parsing code

2015-08-27 Thread Yayoi Ukai
On Sat, Aug 15, 2015 at 11:08 AM, Clément Bœsch wrote: > On Sat, Aug 08, 2015 at 12:52:04PM -0700, Yayoi Ukai wrote: > [...] >> >> -while (dst->len >= 2 && !strncmp(&dst->str[dst->len - 2], "\\N", 2)) >> >> -dst->len -= 2; >> >> -dst->str[dst->len] = 0; >> >> -rstrip_spaces_buf

[FFmpeg-devel] [PATCH 4/5] avformat/samidec:do not include trailing sami

2015-08-27 Thread Yayoi
footer in the last packet --- libavformat/samidec.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavformat/samidec.c b/libavformat/samidec.c index 948e1ed..bc9b745 100644 --- a/libavformat/samidec.c +++ b/libavformat/samidec.c @@ -68,11 +68,17 @@ static int sami_rea

[FFmpeg-devel] [PATCH 1/5] avcodec/srtdec: factor out HTML parsing code

2015-08-27 Thread Yayoi
This code will be used in SAMI decoder in a later commit. --- libavcodec/Makefile| 2 +- libavcodec/htmlsubtitles.c | 177 + libavcodec/htmlsubtitles.h | 29 libavcodec/srtdec.c| 149 +- 4 f

[FFmpeg-devel] [PATCH 5/5] fate/subtitles: add a new test for SAMI demuxer

2015-08-27 Thread Yayoi
and decoder --- tests/fate/subtitles.mak | 3 ++ tests/ref/fate/sub-sami2 | 91 2 files changed, 94 insertions(+) create mode 100644 tests/ref/fate/sub-sami2 diff --git a/tests/fate/subtitles.mak b/tests/fate/subtitles.mak index d8b2034..f5270ea

[FFmpeg-devel] [PATCH 3/5] libavcodec/htmlsubtitles:Add parsing for sami

2015-08-27 Thread Yayoi
--- libavcodec/htmlsubtitles.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/htmlsubtitles.c b/libavcodec/htmlsubtitles.c index a138955..c1324d6 100644 --- a/libavcodec/htmlsubtitles.c +++ b/libavcodec/htmlsubtitles.c @@ -54,7 +54,7 @@ static void rstrip_sp

[FFmpeg-devel] [PATCH 2/5] avcodec/samidec: use ff_htmlmarkup_to_ass()

2015-08-27 Thread Yayoi
--- libavcodec/Makefile | 2 +- libavcodec/samidec.c| 35 +-- tests/ref/fate/sub-sami | 18 +- 3 files changed, 31 insertions(+), 24 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index d1ffb12..1045d7a 100644 --- a/lib

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-27 Thread Robert Krüger
On Fri, Aug 28, 2015 at 12:01 AM, Carl Eugen Hoyos wrote: > Robert Krüger lesspain.de> writes: > > > > > you are required to deinterlace before this > > > > filter and re-interlace after this filter. > > > > > > Shouldn't this be: > > > ... required to deinterleave before and interleave > > > af

Re: [FFmpeg-devel] [PATCH]Mention webm default codec change in the Changelog

2015-08-27 Thread Carl Eugen Hoyos
Thilo Borgmann mail.de> writes: > Am 27.08.15 um 10:51 schrieb Carl Eugen Hoyos: > > Hi! > > > > Better wording welcome! > > Switched default encoders for webm to VP9 and Opus Pushed with this wording. Thank you, Carl Eugen ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] configure: colorize warning messages

2015-08-27 Thread Ganesh Ajjanagadde
On Thu, Aug 27, 2015 at 11:02 PM, Ganesh Ajjanagadde wrote: > On Thu, Aug 27, 2015 at 7:23 PM, Timothy Gu wrote: >> On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote: >>> This patch introduces color warning messages (yellow). >>> Tested with zsh, bash, and sh (bash under --posix)

[FFmpeg-devel] [PATCHv2] configure: colorize warning messages

2015-08-27 Thread Ganesh Ajjanagadde
This patch introduces color warning messages (yellow). They are enabled whenever available. Tested with zsh, bash, and sh (bash under --posix). Signed-off-by: Ganesh Ajjanagadde --- configure | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/configure b/configure index

Re: [FFmpeg-devel] [PATCH] configure: colorize warning messages

2015-08-27 Thread Ganesh Ajjanagadde
On Thu, Aug 27, 2015 at 7:23 PM, Timothy Gu wrote: > On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote: >> This patch introduces color warning messages (yellow). >> Tested with zsh, bash, and sh (bash under --posix). >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> configure |

Re: [FFmpeg-devel] [PATCH] apng: Support inter-frame compression

2015-08-27 Thread Michael Niedermayer
On Wed, Aug 19, 2015 at 06:41:23AM +, Donny Yang wrote: > The current algorithm is just "try all the combinations, and pick the best". > It's not very fast either, probably due to a lot of copying, but will do for > an initial implementation. > > Signed-off-by: Donny Yang > --- > libavcodec/

[FFmpeg-devel] ffplay buffer

2015-08-27 Thread Dan Gordon
I am trying to reduce the size of the buffer for ffplay, it looks like the default is around 500 ms. I have been looking through the code but unfortunately, I’m struggling with understanding it as I’m not too familiar with c yet. Can somebody point me in the right direction on how to reduce this

Re: [FFmpeg-devel] [PATCH] lavf/http: implement directory listing callbacks for Apache

2015-08-27 Thread Mariusz Szczepańczyk
On Mon, Aug 24, 2015 at 12:39 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Fri, Aug 21, 2015 at 01:17:20AM +0200, Mariusz Szczepańczyk wrote: > [...] > > > +static int http_open_dir(URLContext *h) > > +{ > > +HTTPContext *s = h->priv_data; > > +xmlSAXHandler handlers = {};

Re: [FFmpeg-devel] [PATCH 2/2] lavf/file: check for dirent.h support

2015-08-27 Thread Mariusz Szczepańczyk
On Tue, Aug 25, 2015 at 2:36 PM, Michael Niedermayer wrote: > On Tue, Aug 25, 2015 at 02:18:22AM -0300, James Almer wrote: > > On 24/08/15 10:02 PM, Michael Niedermayer wrote: > > > On Fri, Aug 21, 2015 at 02:18:34AM +0200, Mariusz Szczepańczyk wrote: > > >> On Thu, Jun 25, 2015 at 12:09 PM, Mich

Re: [FFmpeg-devel] [PATCH] configure: colorize warning messages

2015-08-27 Thread Ganesh Ajjanagadde
sent from my phone On Aug 27, 2015 7:26 PM, "Timothy Gu" wrote: > > On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote: > > This patch introduces color warning messages (yellow). > > Tested with zsh, bash, and sh (bash under --posix). > > > > Signed-off-by: Ganesh Ajjanagadde > >

Re: [FFmpeg-devel] PATCH: better message that ffplay is not going to be built

2015-08-27 Thread Timothy Gu
On Thu, Aug 27, 2015 at 06:36:00PM -0400, Ganesh Ajjanagadde wrote: > Please add the area (e.g configure:...) to the commit message. > More generally, I suggest a slight cleanup of the commit message: > first line-> "configure: print out enabled programs" > skip a blank line, body-> "This prints ou

Re: [FFmpeg-devel] [PATCH] configure: colorize warning messages

2015-08-27 Thread Timothy Gu
On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote: > This patch introduces color warning messages (yellow). > Tested with zsh, bash, and sh (bash under --posix). > > Signed-off-by: Ganesh Ajjanagadde > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) I

Re: [FFmpeg-devel] [PATCH] vaapi: Add hevc hwaccel support

2015-08-27 Thread Timo Rothenpieler
> Hi, > > Sorry, I cannot test until next week. Does it actually work now? Based > on some #ffmpeg-devel backlogs :) Yes, it works flawlessly, except for samples using ScalingLists. See http://fate-suite.ffmpeg.org/hevc-conformance/ the SLIST_* files. Those files decode with strong artifacts, wi

Re: [FFmpeg-devel] PATCH: better message that ffplay is not going to be built

2015-08-27 Thread Ganesh Ajjanagadde
On Thu, Aug 27, 2015 at 6:18 PM, Roger Pack wrote: > See attached. > Basically it was annoying to not be able to tell that ffplay was > "skipped" for some reason (like no SDL) but now you can since it > prints out what it will build. > Thank you. Please add the area (e.g configure:...) to the com

[FFmpeg-devel] PATCH: better message that ffplay is not going to be built

2015-08-27 Thread Roger Pack
See attached. Basically it was annoying to not be able to tell that ffplay was "skipped" for some reason (like no SDL) but now you can since it prints out what it will build. Thank you. 0001-better-message-that-ffplay-is-not-going-to-be-built-.patch Description: Binary data __

[FFmpeg-devel] [PATCH] configure: colorize warning messages

2015-08-27 Thread Ganesh Ajjanagadde
This patch introduces color warning messages (yellow). Tested with zsh, bash, and sh (bash under --posix). Signed-off-by: Ganesh Ajjanagadde --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 8840027..7f43572 100755 --- a/configure ++

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-27 Thread Carl Eugen Hoyos
Robert Krüger lesspain.de> writes: > > > you are required to deinterlace before this > > > filter and re-interlace after this filter. > > > > Shouldn't this be: > > ... required to deinterleave before and interleave > > after this filter. > > ? > > Could you elaborate why? > > I think deinter

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-27 Thread Robert Krüger
Am 27.08.2015 23:24 schrieb "Carl Eugen Hoyos" : > > Paul B Mahol gmail.com> writes: > > > +This filter is not designed to function correctly with interlaced media. If > > +you wish to change the frame rate of interlaced media then you are required > > +to deinterlace before this filter and re-int

Re: [FFmpeg-devel] [PATCH] avfilter: add framerate video filter

2015-08-27 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > +This filter is not designed to function correctly with interlaced media. If > +you wish to change the frame rate of interlaced media then you are required > +to deinterlace before this filter and re-interlace after this filter. Shouldn't this be: ... required t

Re: [FFmpeg-devel] [PATCH] avfilter/af_amerge: use the name 's' for the pointer to the private context

2015-08-27 Thread Paul B Mahol
On 8/26/15, Nicolas George wrote: > Le nonidi 9 fructidor, an CCXXIII, Ganesh Ajjanagadde a ecrit : >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/af_amerge.c | 104 >> >> 1 file changed, 52 insertions(+), 52 deletions(-) > > Ok. appl

Re: [FFmpeg-devel] [PATCH]Describe eia_608 as text subtitles

2015-08-27 Thread compn
On Thu, 27 Aug 2015 19:54:07 +0530 Anshul wrote: > On 08/27/2015 05:46 PM, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes an issue with ffplay here (it tries to scale > > the text subtitles), please comment, > Though patch looks correct to me, just need more information for > testin

Re: [FFmpeg-devel] [PATCH] avfilter/af_amerge: use the name 's' for the pointer to the private context

2015-08-27 Thread Ganesh Ajjanagadde
On Wed, Aug 26, 2015 at 12:55 PM, Nicolas George wrote: > Le nonidi 9 fructidor, an CCXXIII, Ganesh Ajjanagadde a écrit : >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/af_amerge.c | 104 >> >> 1 file changed, 52 insertions(+), 52 del

Re: [FFmpeg-devel] [PATCH] avfilter/vf_histogram: 9 and 10 bit depth support

2015-08-27 Thread Michael Niedermayer
On Wed, Aug 26, 2015 at 09:11:34AM +, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_histogram.c | 136 > +++-- > 1 file changed, 106 insertions(+), 30 deletions(-) LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF21

Re: [FFmpeg-devel] [PATCH] vaapi: Add hevc hwaccel support

2015-08-27 Thread Gwenole Beauchesne
Hi, Sorry, I cannot test until next week. Does it actually work now? Based on some #ffmpeg-devel backlogs :) 2015-08-26 15:15 GMT+02:00 Timo Rothenpieler : > --- > configure | 4 + > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/hevc.c |

Re: [FFmpeg-devel] [PATCH] ffplay: remove unused include libavutil/colorspace.h

2015-08-27 Thread Michael Niedermayer
On Thu, Aug 27, 2015 at 02:00:30PM +0800, Zhang Rui wrote: > --- > ffplay.c | 1 - > 1 file changed, 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB DNS cache poisoning attacks, popular search engine, Google internet authority dont

Re: [FFmpeg-devel] [PATCH] lavf/mpegenc: provide better feedback in case of invalid media type

2015-08-27 Thread Stefano Sabatini
On date Thursday 2015-08-27 17:59:20 +0200, Michael Niedermayer wrote: > On Thu, Aug 27, 2015 at 05:42:55PM +0200, Stefano Sabatini wrote: > > --- > > libavformat/mpegenc.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/libavformat/mpegenc.c b/libavformat/mpegen

Re: [FFmpeg-devel] [PATCH] lavf/mpegenc: provide better feedback in case of invalid media type

2015-08-27 Thread Michael Niedermayer
On Thu, Aug 27, 2015 at 05:42:55PM +0200, Stefano Sabatini wrote: > --- > libavformat/mpegenc.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c > index 2520f49..b3ee2a0 100644 > --- a/libavformat/mpegenc.c > +++ b/libavform

Re: [FFmpeg-devel] [PATCH] QSV : Added look ahead rate control mode

2015-08-27 Thread Michael Niedermayer
On Thu, Aug 27, 2015 at 11:02:44AM +0200, Sven Dueking wrote: > > -Ursprüngliche Nachricht- > > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > > von Sven Dueking > > Gesendet: Freitag, 21. August 2015 10:18 > > An: ffmpeg-devel@ffmpeg.org > > Cc: Sven Dueking > > Be

[FFmpeg-devel] [PATCH] lavf/mpegenc: provide better feedback in case of invalid media type

2015-08-27 Thread Stefano Sabatini
--- libavformat/mpegenc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c index 2520f49..b3ee2a0 100644 --- a/libavformat/mpegenc.c +++ b/libavformat/mpegenc.c @@ -410,7 +410,9 @@ static av_cold int mpeg_mux_init(AVFormatContext

Re: [FFmpeg-devel] [PATCH] avcodec: Assert on codec->encode2 in encode_audio2

2015-08-27 Thread Michael Niedermayer
On Thu, Aug 27, 2015 at 12:40:30PM +0200, lummax wrote: > Assert on `avctx->codec->encode2` to avoid a SEGFAULT on the subsequent > function call. > --- > libavcodec/utils.c | 2 ++ > 1 file changed, 2 insertions(+) commit messages extended applied thanks [...] -- Michael GnuPG fingerprint

Re: [FFmpeg-devel] [PATCHv3] configure: warn if GCC 4.2 is being used

2015-08-27 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde gmail.com> writes: > The wiki, Ticket1464, and Ticket3970 warn about the > usage of GCC 4.2. > This fixes Ticket3970. Pushed. Thank you for working on this issue, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org ht

Re: [FFmpeg-devel] [PATCH]Describe eia_608 as text subtitles

2015-08-27 Thread Carl Eugen Hoyos
Anshul gmail.com> writes: > > Attached patch fixes an issue with ffplay here (it > > tries to scale the text subtitles), please comment, > > Any Video sample that work with this patch and do > not work without it would be great. Full Episode 2.mp4 > At my PC I am not able to play any ClosedC

Re: [FFmpeg-devel] [PATCH]Describe eia_608 as text subtitles

2015-08-27 Thread Anshul
On 08/27/2015 05:46 PM, Carl Eugen Hoyos wrote: Hi! Attached patch fixes an issue with ffplay here (it tries to scale the text subtitles), please comment, Any Video sample that work with this patch and do not work without it would be great. At my PC I am not able to play any ClosedCaption

[FFmpeg-devel] [PATCH]Describe eia_608 as text subtitles

2015-08-27 Thread Carl Eugen Hoyos
Hi! Attached patch fixes an issue with ffplay here (it tries to scale the text subtitles), please comment, Carl Eugen diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c index 729e855..c172bf8 100644 --- a/libavcodec/codec_desc.c +++ b/libavcodec/codec_desc.c @@ -2670,6 +2670,7 @@ sta

Re: [FFmpeg-devel] [PATCH] avcodec: Assert on codec->encode2 in encode_audio2

2015-08-27 Thread wm4
On Thu, 27 Aug 2015 12:55:17 +0200 Lukas Blubb wrote: > 2015-08-27 12:45 GMT+02:00 wm4 : > > [..] > > > > When can this happen? Shouldn't it just return an error somewhere if > > this is not an encoder? > > I'm working on a Rust wrapper library and apparently am doing something > wrong (it segfa

Re: [FFmpeg-devel] [PATCH] avcodec: Assert on codec->encode2 in encode_audio2

2015-08-27 Thread Lukas Blubb
2015-08-27 12:45 GMT+02:00 wm4 : > [..] > > When can this happen? Shouldn't it just return an error somewhere if > this is not an encoder? I'm working on a Rust wrapper library and apparently am doing something wrong (it segfaulted). `avcodec_encode_video2()` has this assertion, so I thought it mi

Re: [FFmpeg-devel] [PATCH] avcodec: Assert on codec->encode2 in encode_audio2

2015-08-27 Thread wm4
On Thu, 27 Aug 2015 12:40:30 +0200 lummax wrote: > Assert on `avctx->codec->encode2` to avoid a SEGFAULT on the subsequent > function call. > --- > libavcodec/utils.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 4da16ee..f51070c 1006

[FFmpeg-devel] [PATCH] avcodec: Assert on codec->encode2 in encode_audio2

2015-08-27 Thread lummax
Assert on `avctx->codec->encode2` to avoid a SEGFAULT on the subsequent function call. --- libavcodec/utils.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 4da16ee..f51070c 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -1953,6 +19

Re: [FFmpeg-devel] [PATCH] configure: remove unused apply() function

2015-08-27 Thread Michael Niedermayer
On Thu, Aug 27, 2015 at 01:08:34AM -0400, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > configure | 6 -- > 1 file changed, 6 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay dictionary, pag

Re: [FFmpeg-devel] [PATCH]Mention webm default codec change in the Changelog

2015-08-27 Thread Thilo Borgmann
Am 27.08.15 um 10:51 schrieb Carl Eugen Hoyos: > Hi! > > Better wording welcome! Switched default encoders for webm to VP9 and Opus -Thilo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Connection timeout to HTTP resource

2015-08-27 Thread Przemysław Sobala
W dniu 26.08.2015 o 18:27, Stephan Holljes pisze: Hi, On Wed, Aug 26, 2015 at 11:35 AM, Przemysław Sobala wrote: Hi I wanted to configure tcp connection timeout while connecting to eg. http://google.com:81. ffmpeg -i http://google.com:81?timeout=100 doesn't pass "timeout" param to tcp pro

Re: [FFmpeg-devel] [PATCH] QSV : Added look ahead rate control mode

2015-08-27 Thread Sven Dueking
> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Sven Dueking > Gesendet: Freitag, 21. August 2015 10:18 > An: ffmpeg-devel@ffmpeg.org > Cc: Sven Dueking > Betreff: [FFmpeg-devel] [PATCH] QSV : Added look ahead rate control > mode >

[FFmpeg-devel] [PATCH]Mention webm default codec change in the Changelog

2015-08-27 Thread Carl Eugen Hoyos
Hi! Better wording welcome! Thank you, Carl Eugen diff --git a/Changelog b/Changelog index 844f5a5..9de156b 100644 --- a/Changelog +++ b/Changelog @@ -36,6 +36,7 @@ version : - waveform filter - hstack and vstack filter - Support DNx100 (1440x1080@8) +- VP9 and Opus new default codecs when enc