Re: [FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

2015-12-17 Thread Paul B Mahol
On 12/17/15, Mats Peterson wrote: > On 12/16/2015 08:30 PM, Mats Peterson wrote: >> On 12/16/2015 08:27 PM, Mats Peterson wrote: >>> On 12/16/2015 08:26 PM, Kieran Kunhya wrote: > I have said before that I'm not going to dwell into the complexities > of

Re: [FFmpeg-devel] [PATCH] on2avc: limit number of bits to 30 in get_egolomb

2015-12-17 Thread Hendrik Leppkes
On Wed, Dec 16, 2015 at 8:20 PM, Andreas Cadhalpun wrote: > More don't fit into the integer output. > > Also use get_bits_long, since get_bits only supports reading up to 25 > bits, while get_bits_long supports the full integer range. > > Signed-off-by: Andreas

Re: [FFmpeg-devel] [PATCH] Revert "Merge commit '741b494fa8cd28a7d096349bac183893c236e3f9'"

2015-12-17 Thread Ronald S. Bultje
Hi, On Thu, Dec 17, 2015 at 8:46 AM, Hendrik Leppkes wrote: > Reverting big commits to fix one sample and making future merges > harder or impossible seems not like the best option, unless it can be > shown that the entire change is flawed, and not just a small bug. >

Re: [FFmpeg-devel] [PATCH] lavc: add text encoder

2015-12-17 Thread Moritz Barsnick
On Thu, Dec 17, 2015 at 13:56:57 +0100, Clément Bœsch wrote: > It's not actually "srt"/subrip at all since the packet payload will only > contain a text string with no markup, while subrip contains the text > string with the html tag markup. Ah, that's the part about SRT ("formally" vs. text vs.

Re: [FFmpeg-devel] [PATCH] Revert "Merge commit '741b494fa8cd28a7d096349bac183893c236e3f9'"

2015-12-17 Thread Hendrik Leppkes
On Thu, Dec 17, 2015 at 2:06 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > This fixes a regression of the sample from Ticket 2371 > > This reverts commit bc66451e5e903698ee0500faf04c1214f3dd157f, reversing > changes made to

Re: [FFmpeg-devel] [PATCH] avcodec/avpacket: Replace EOVERFLOW by EINVAL

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 03:18:12PM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > EOVERFLOW is not available on all platforms disregard this, i wrote this before seeing 68e547ae8b455e5e2b60839f35c359d77a6d94bc [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] Add "xmv" extension to the xmv codec

2015-12-17 Thread Moritz Barsnick
On Thu, Dec 17, 2015 at 12:31:20 +, Carl Eugen Hoyos wrote: > > I just submitted a pull request, #166, to > This is not the preferred way of contributing, please > consider to send the patch here. Which shows that it's time to push this change of the docs:

[FFmpeg-devel] [PATCH] avcodec/avpacket: Replace EOVERFLOW by EINVAL

2015-12-17 Thread Michael Niedermayer
From: Michael Niedermayer EOVERFLOW is not available on all platforms See: dec72f8826e3f2032aec463c3dd1e26ec2a4 Signed-off-by: Michael Niedermayer --- libavcodec/avpacket.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH] Revert "Merge commit '741b494fa8cd28a7d096349bac183893c236e3f9'"

2015-12-17 Thread Michael Niedermayer
From: Michael Niedermayer This fixes a regression of the sample from Ticket 2371 This reverts commit bc66451e5e903698ee0500faf04c1214f3dd157f, reversing changes made to 9d1fb9ef313e0fb709ac4c35c7bf00264963fd85. --- libavcodec/h264.h |8 +++

[FFmpeg-devel] [PATCH] lavu/intmath: add faster clz support

2015-12-17 Thread Ganesh Ajjanagadde
This should be useful for the sofalizer filter. Signed-off-by: Ganesh Ajjanagadde --- libavutil/intmath.h | 18 ++ 1 file changed, 18 insertions(+) diff --git a/libavutil/intmath.h b/libavutil/intmath.h index 2016723..2bb0b62 100644 ---

Re: [FFmpeg-devel] [PATCH] Some minor fixes of my "patchset" for palettized QuickTime video in Matroska

2015-12-17 Thread Mats Peterson
On 12/17/2015 04:35 PM, Mats Peterson wrote: I know nobody will care whatsoever, but I forgot to change the minimum private data size from 86 to 36 for A_QUICKTIME, and I removed that fprintf() line in qtpalette.c. I'll post it for the hell of it. PERHAPS it can be valuable as a starting point

Re: [FFmpeg-devel] [PATCH 2/2] swscale/arm/yuv2rgb: add ff_yuv420p_to_{argb, rgba, abgr, bgra}_neon_{16, 32}

2015-12-17 Thread Matthieu Bouron
On Tue, Dec 15, 2015 at 06:22:43PM +0100, Michael Niedermayer wrote: > On Tue, Dec 15, 2015 at 05:46:09PM +0100, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > --- > > > > Hi, > > > > This commit is likely to break fate on arm since the current C code

Re: [FFmpeg-devel] [PATCH] Some minor fixes of my "patchset" for palettized QuickTime video in Matroska

2015-12-17 Thread Mats Peterson
On 12/17/2015 04:56 PM, Paul B Mahol wrote: Dana 17. 12. 2015. 16:51 osoba "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org> napisala je: On 12/17/2015 04:35 PM, Mats Peterson wrote: I know nobody will care whatsoever, but I forgot to change the minimum private data size from 86 to 36 for

[FFmpeg-devel] [PATCH] avcodec/opus_celt: replace pow by exp2

2015-12-17 Thread Ganesh Ajjanagadde
exp2 is faster. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/opus_celt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/opus_celt.c b/libavcodec/opus_celt.c index 9911de3..474452f 100644 --- a/libavcodec/opus_celt.c +++

Re: [FFmpeg-devel] [PATCH] avcodec/nellymoserenc: avoid wasteful pow

2015-12-17 Thread Ganesh Ajjanagadde
On Tue, Dec 15, 2015 at 6:40 PM, Ganesh Ajjanagadde wrote: > On Tue, Dec 15, 2015 at 5:25 PM, Ganesh Ajjanagadde wrote: >> On Tue, Dec 15, 2015 at 2:23 AM, Michael Niedermayer >> wrote: >>> On Wed, Dec 09, 2015 at 06:55:25PM -0500, Ganesh

Re: [FFmpeg-devel] [PATCH 1/2] avutil/random_seed: use arc4random() when available

2015-12-17 Thread Ganesh Ajjanagadde
On Mon, Dec 14, 2015 at 8:05 AM, Ganesh Ajjanagadde wrote: > On Thu, Dec 10, 2015 at 3:31 PM, Ganesh Ajjanagadde > wrote: >> On Wed, Dec 9, 2015 at 8:08 AM, Ganesh Ajjanagadde >> wrote: >>> On Sun, Dec 6, 2015 at 10:56 PM,

Re: [FFmpeg-devel] [PATCH] Some minor fixes of my "patchset" for palettized QuickTime video in Matroska

2015-12-17 Thread Paul B Mahol
Dana 17. 12. 2015. 16:51 osoba "Mats Peterson" < matsp888-at-yahoo@ffmpeg.org> napisala je: > > On 12/17/2015 04:35 PM, Mats Peterson wrote: >> >> I know nobody will care whatsoever, but I forgot to change the minimum >> private data size from 86 to 36 for A_QUICKTIME, and I removed that >>

Re: [FFmpeg-devel] [PATCH] lavu/intmath: add faster clz support

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 8:33 AM, Ganesh Ajjanagadde wrote: > This should be useful for the sofalizer filter. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/intmath.h | 18 ++ > 1 file changed, 18 insertions(+) > > diff

[FFmpeg-devel] [PATCH 2/2] avformat/oggparsedirac: Export sample aspect ratio

2015-12-17 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavformat/oggparsedirac.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/oggparsedirac.c b/libavformat/oggparsedirac.c index a39ea4a..3e5e393 100644 ---

[FFmpeg-devel] [PATCH 1/2] avcodec/diracdec: fix aspect ratio (it was lost after efcc8fddd6b7d1f931ff349e195d78c3c943d7fd)

2015-12-17 Thread Michael Niedermayer
From: Michael Niedermayer Signed-off-by: Michael Niedermayer --- libavcodec/diracdec.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/diracdec.c b/libavcodec/diracdec.c index 000abf7..f35fe45 100644 --- a/libavcodec/diracdec.c

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/diracdec: fix aspect ratio (it was lost after efcc8fddd6b7d1f931ff349e195d78c3c943d7fd)

2015-12-17 Thread Hendrik Leppkes
On Thu, Dec 17, 2015 at 4:13 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavcodec/diracdec.c |1 + > 1 file changed, 1 insertion(+) > > diff --git

Re: [FFmpeg-devel] [PATCH] avfilter: add ITU-R 468 noise meter

2015-12-17 Thread Ganesh Ajjanagadde
On Wed, Dec 16, 2015 at 1:34 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol [...] > + > +formats = ff_make_format_list(sample_fmts); > +if ((ret = ff_formats_ref(formats, >out_formats)) < 0 || > +(ret = ff_formats_ref(formats,

Re: [FFmpeg-devel] [PATCH] lavu/intmath: add faster clz support

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 9:25 AM, Clément Bœsch wrote: > On Thu, Dec 17, 2015 at 09:23:53AM -0800, Ganesh Ajjanagadde wrote: >> On Thu, Dec 17, 2015 at 9:20 AM, Kieran Kunhya wrote: >> >>> +static av_always_inline av_const unsigned ff_clz_c(unsigned v) >> >>> +{ >>

[FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Rostislav Pehlivanov
This commit adds a raw integer log2 function (which directly maps to a builtin). This was needed as the Daala entropy coding system requires a raw log2 and the existing ff_log2 OR'd the argument with 1. The C implementation was taken from Daala's source code, which got it from someplace else as a

Re: [FFmpeg-devel] [PATCH v2] avcodec/nvenc: Include NVENC SDK header

2015-12-17 Thread Timo Rothenpieler
> But before doing any of this, there has to be an agreement, whether or not > the header should be included at all. Of course. Here's what I intend to do once I find some time again, which could take a few days/weeks: Split this up into three patches. One that adds a check for NVENC Version 6,

Re: [FFmpeg-devel] [PATCH 2/2] swscale/arm/yuv2rgb: add ff_yuv420p_to_{argb, rgba, abgr, bgra}_neon_{16, 32}

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 04:54:31PM +0100, Matthieu Bouron wrote: > On Tue, Dec 15, 2015 at 06:22:43PM +0100, Michael Niedermayer wrote: > > On Tue, Dec 15, 2015 at 05:46:09PM +0100, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > > > --- > > > > > > Hi,

Re: [FFmpeg-devel] [PATCH] lavu/intmath: add faster clz support

2015-12-17 Thread Kieran Kunhya
>> +static av_always_inline av_const unsigned ff_clz_c(unsigned v) >> +{ >> +unsigned i = sizeof(x) * 8; >> + >> +while (x) { >> +x >>= 1; >> +i--; >> +} >> + >> +return i; >> +} >> +#endif >> + erm, does even work? ___

Re: [FFmpeg-devel] [PATCH] lavu/intmath: add faster clz support

2015-12-17 Thread Clément Bœsch
On Thu, Dec 17, 2015 at 09:23:53AM -0800, Ganesh Ajjanagadde wrote: > On Thu, Dec 17, 2015 at 9:20 AM, Kieran Kunhya wrote: > >>> +static av_always_inline av_const unsigned ff_clz_c(unsigned v) > >>> +{ > >>> +unsigned i = sizeof(x) * 8; > >>> + > >>> +while (x) { > >>> +

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Carl Eugen Hoyos
Rostislav Pehlivanov gmail.com> writes: > +/* From Daala, src/internal.c, od_ilog() */ How does the copyright statement and license in internal.c look like? > +static av_always_inline av_const int > ff_log2_c_nz(unsigned int x) Shouldn't this be uint32_t (and uint64_t below)? Carl Eugen

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/diracdec: fix aspect ratio (it was lost after efcc8fddd6b7d1f931ff349e195d78c3c943d7fd)

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 05:26:13PM +0100, Hendrik Leppkes wrote: > On Thu, Dec 17, 2015 at 4:13 PM, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > Signed-off-by: Michael Niedermayer > > --- > >

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 05:29:58PM +, Rostislav Pehlivanov wrote: > This commit adds a raw integer log2 function (which directly maps to a > builtin). This was needed as the Daala entropy coding system requires a > raw log2 and the existing ff_log2 OR'd the argument with 1. __builtin_clz(0)

Re: [FFmpeg-devel] [PATCH] lavu/intmath: add faster clz support

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 9:20 AM, Kieran Kunhya wrote: >>> +static av_always_inline av_const unsigned ff_clz_c(unsigned v) >>> +{ >>> +unsigned i = sizeof(x) * 8; >>> + >>> +while (x) { >>> +x >>= 1; >>> +i--; >>> +} >>> + >>> +return i; >>> +} >>>

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 9:29 AM, Rostislav Pehlivanov wrote: > This commit adds a raw integer log2 function (which directly maps to a > builtin). This was needed as the Daala entropy coding system requires a > raw log2 and the existing ff_log2 OR'd the argument with 1. > >

Re: [FFmpeg-devel] [PATCH] sonic: make sure num_taps is not larger than frame_size

2015-12-17 Thread Andreas Cadhalpun
On 16.12.2015 21:42, Michael Niedermayer wrote: > On Wed, Dec 16, 2015 at 07:27:05PM +0100, Andreas Cadhalpun wrote: >> sonic.c |7 +++ >> 1 file changed, 7 insertions(+) >> cfe08ddd0c3dcae138abec66e1e08e628a88b41a >> 0001-sonic-make-sure-num_taps-channels-is-not-larger-than.patch >>

Re: [FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

2015-12-17 Thread Lou Logan
On Thu, 17 Dec 2015 07:58:27 +0100, Mats Peterson wrote: > And based on Michael Niedermayer's statement regarding the goal of > having FFmpeg "support all kinds of odd and broken files", why aren't > you people more interested in fixing this long-standing issue regarding > palettized QuickTime

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Rostislav Pehlivanov
On Thu, 2015-12-17 at 19:22 +0100, Michael Niedermayer wrote: > On Thu, Dec 17, 2015 at 05:29:58PM +, Rostislav Pehlivanov wrote: > > This commit adds a raw integer log2 function (which directly maps > > to a > > builtin). This was needed as the Daala entropy coding system > > requires a > >

Re: [FFmpeg-devel] [GSoC] BDA (DTV) Capture / tuning -- work-in-progress

2015-12-17 Thread Roger Pack
On 8/22/15, Michael Niedermayer wrote: > On Sat, Aug 22, 2015 at 05:18:04AM +0200, Máté Sebők wrote: >> > >> > fails to build with mingw due to missing header >> > In file included from ffmpeg/libavdevice/dshow.c:33:0: >> > ffmpeg/libavdevice/bdadefs.h:8:24: fatal error:

Re: [FFmpeg-devel] [PATCH] avcodec/opus_celt: replace pow by exp2

2015-12-17 Thread Paul B Mahol
On 12/17/15, Ganesh Ajjanagadde wrote: > exp2 is faster. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/opus_celt.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/opus_celt.c

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Rostislav Pehlivanov
On Thu, 2015-12-17 at 22:17 +0100, Michael Niedermayer wrote: > > 32 and (32|1) have the same count of leading zeros so both functions > would return the same value, what am i missing ? Damn, sorry, I think I was half asleep while writing the patch. Probably because I forgot to add 1 to

Re: [FFmpeg-devel] [PATCH] on2avc: limit number of bits to 30 in get_egolomb

2015-12-17 Thread Andreas Cadhalpun
On 17.12.2015 10:54, Hendrik Leppkes wrote: > On Wed, Dec 16, 2015 at 8:20 PM, Andreas Cadhalpun > wrote: >> More don't fit into the integer output. >> >> Also use get_bits_long, since get_bits only supports reading up to 25 >> bits, while get_bits_long supports

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 1:58 PM, Rostislav Pehlivanov wrote: > On Thu, 2015-12-17 at 22:17 +0100, Michael Niedermayer wrote: >> >> 32 and (32|1) have the same count of leading zeros so both functions >> would return the same value, what am i missing ? > > Damn, sorry, I think

Re: [FFmpeg-devel] [PATCH] avcodec/opus_celt: replace pow by exp2

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 11:10 AM, Paul B Mahol wrote: > On 12/17/15, Ganesh Ajjanagadde wrote: >> exp2 is faster. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/opus_celt.c | 4 ++-- >> 1 file changed, 2

Re: [FFmpeg-devel] [PATCH] intmath.h: add an ff_log2_nz

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 08:44:38PM +, Rostislav Pehlivanov wrote: > On Thu, 2015-12-17 at 19:22 +0100, Michael Niedermayer wrote: > > On Thu, Dec 17, 2015 at 05:29:58PM +, Rostislav Pehlivanov wrote: > > > This commit adds a raw integer log2 function (which directly maps > > > to a > > >

Re: [FFmpeg-devel] [PATCH] nuv: reject negative fps rate

2015-12-17 Thread Andreas Cadhalpun
On 16.12.2015 23:53, Michael Niedermayer wrote: > On Wed, Dec 16, 2015 at 08:56:55PM +0100, Andreas Cadhalpun wrote: >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/nuv.c | 4 >> 1 file changed, 4 insertions(+) > > looks logical iam not nuv

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: add flag to embed an AC-3/E-AC-3 ES the DVB way

2015-12-17 Thread Stefan Pöschel
*bump* Am 13.12.2015 um 15:23 schrieb Stefan Pöschel: > Am 13.12.2015 um 14:18 schrieb Moritz Barsnick: >> On Sun, Dec 13, 2015 at 11:54:32 +0100, Stefan Pöschel wrote: >>> So far an AC-3 elementary stream is refered to in the PMT according to >>> System A (ATSC). An E-AC-3 ES in contrast is

Re: [FFmpeg-devel] [PATCH] on2avc: limit number of bits to 30 in get_egolomb

2015-12-17 Thread Andreas Cadhalpun
On 17.12.2015 13:28, Michael Niedermayer wrote: > On Wed, Dec 16, 2015 at 08:20:18PM +0100, Andreas Cadhalpun wrote: >> More don't fit into the integer output. >> >> Also use get_bits_long, since get_bits only supports reading up to 25 >> bits, while get_bits_long supports the full integer range.

[FFmpeg-devel] [PATCH] lavf/avformat: add av_warn_unused_result to avformat_write_header

2015-12-17 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavformat/avformat.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index ddf07b1..32bed01 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -2283,6 +2283,7 @@

Re: [FFmpeg-devel] [libav-devel] [PATCH] nuv: reject negative fps rate

2015-12-17 Thread Andreas Cadhalpun
On 16.12.2015 22:18, Luca Barbato wrote: > On 16/12/15 20:56, Andreas Cadhalpun wrote: >> Signed-off-by: Andreas Cadhalpun >> --- >> libavformat/nuv.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/libavformat/nuv.c b/libavformat/nuv.c >> index

Re: [FFmpeg-devel] [libav-devel] [PATCH] on2avc: limit number of bits to 30 in get_egolomb

2015-12-17 Thread Andreas Cadhalpun
On 16.12.2015 22:26, Luca Barbato wrote: > On 16/12/15 20:20, Andreas Cadhalpun wrote: >> More don't fit into the integer output. >> >> Also use get_bits_long, since get_bits only supports reading up to 25 >> bits, while get_bits_long supports the full integer range. >> >> Signed-off-by: Andreas

Re: [FFmpeg-devel] [PATCH] ffplay: remove existing AVPicture usage

2015-12-17 Thread Ganesh Ajjanagadde
On Thu, Dec 17, 2015 at 7:56 PM, Marton Balint wrote: > It is deprecated. > > Signed-off-by: Marton Balint > --- > ffplay.c | 51 +++ > 1 file changed, 27 insertions(+), 24 deletions(-) > > diff --git a/ffplay.c

Re: [FFmpeg-devel] [PATCH] nuv: reject negative fps rate

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 11:03:15PM +0100, Andreas Cadhalpun wrote: > On 16.12.2015 23:53, Michael Niedermayer wrote: > > On Wed, Dec 16, 2015 at 08:56:55PM +0100, Andreas Cadhalpun wrote: > >> Signed-off-by: Andreas Cadhalpun > >> --- > >> libavformat/nuv.c | 4

Re: [FFmpeg-devel] [PATCH] lavc: add text encoder

2015-12-17 Thread Moritz Barsnick
On Thu, Dec 17, 2015 at 12:30:27 +0100, Clément Bœsch wrote: > --- > libavcodec/Makefile| 1 + > libavcodec/allcodecs.c | 2 +- > libavcodec/srtenc.c| 37 +++- > libavcodec/version.h | 2 +- > tests/fate/subtitles.mak | 3 + > tests/ref/fate/sub-textenc |

Re: [FFmpeg-devel] [PATCH] on2avc: limit number of bits to 30 in get_egolomb

2015-12-17 Thread Michael Niedermayer
On Wed, Dec 16, 2015 at 08:20:18PM +0100, Andreas Cadhalpun wrote: > More don't fit into the integer output. > > Also use get_bits_long, since get_bits only supports reading up to 25 > bits, while get_bits_long supports the full integer range. > > Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] Add "xmv" extension to the xmv codec

2015-12-17 Thread Carl Eugen Hoyos
Arne Schwettmann gmail.com> writes: > I just submitted a pull request, #166, to > https://github.com/FFmpeg/FFmpeg This is not the preferred way of contributing, please consider to send the patch here. > I am not a ffmpeg developer, but I came across this > problem that the extension "xmv"

[FFmpeg-devel] Vertical bands with RT21 (Intel Real Time Video 2.1) codec

2015-12-17 Thread Mats Peterson
Samples: https://drive.google.com/open?id=0B3_pEBoLs0faei1Sd1h6SG11QlE Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] doc/errno: fix description typo for ENAMETOOLONG

2015-12-17 Thread Marton Balint
On Wed, 9 Dec 2015, Marton Balint wrote: Signed-off-by: Marton Balint --- doc/errno.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/errno.txt b/doc/errno.txt index 31cab26..933a4de 100644 --- a/doc/errno.txt +++ b/doc/errno.txt @@ -76,7 +76,7 @@

Re: [FFmpeg-devel] [PATCH 2/2] lavu/error: add missing error messages for errors supported on all platforms

2015-12-17 Thread Marton Balint
On Sat, 12 Dec 2015, Michael Niedermayer wrote: On Wed, Dec 09, 2015 at 10:12:52PM +0100, Marton Balint wrote: We need these if we have no strerror_r. Descriptions are taken from doc/errno.txt except for ENOMEM. Signed-off-by: Marton Balint --- libavutil/error.c | 37

[FFmpeg-devel] [PATCH] ffplay: remove existing AVPicture usage

2015-12-17 Thread Marton Balint
It is deprecated. Signed-off-by: Marton Balint --- ffplay.c | 51 +++ 1 file changed, 27 insertions(+), 24 deletions(-) diff --git a/ffplay.c b/ffplay.c index 36da8a5..3baefcc 100644 --- a/ffplay.c +++ b/ffplay.c @@ -858,7 +858,7

[FFmpeg-devel] [PATCH] lavc: add text encoder

2015-12-17 Thread Clément Bœsch
--- libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 2 +- libavcodec/srtenc.c| 37 +++- libavcodec/version.h | 2 +- tests/fate/subtitles.mak | 3 + tests/ref/fate/sub-textenc | 213 + 6 files changed, 253

Re: [FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

2015-12-17 Thread Mats Peterson
On 12/17/2015 09:31 AM, Paul B Mahol wrote: On 12/17/15, Mats Peterson wrote: On 12/16/2015 08:30 PM, Mats Peterson wrote: On 12/16/2015 08:27 PM, Mats Peterson wrote: On 12/16/2015 08:26 PM, Kieran Kunhya wrote: I have said before that I'm not going to

Re: [FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

2015-12-17 Thread Mats Peterson
On 12/17/2015 11:53 AM, Mats Peterson wrote: On 12/17/2015 09:31 AM, Paul B Mahol wrote: On 12/17/15, Mats Peterson wrote: On 12/16/2015 08:30 PM, Mats Peterson wrote: On 12/16/2015 08:27 PM, Mats Peterson wrote: On 12/16/2015 08:26 PM, Kieran Kunhya wrote:

Re: [FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

2015-12-17 Thread Paul B Mahol
On 12/17/15, Mats Peterson wrote: > On 12/17/2015 11:53 AM, Mats Peterson wrote: >> On 12/17/2015 09:31 AM, Paul B Mahol wrote: >>> On 12/17/15, Mats Peterson wrote: On 12/16/2015 08:30 PM, Mats Peterson wrote: > On

Re: [FFmpeg-devel] [PATCH] on2avc: limit number of bits to 30 in get_egolomb

2015-12-17 Thread Michael Niedermayer
On Thu, Dec 17, 2015 at 11:30:01PM +0100, Andreas Cadhalpun wrote: > On 17.12.2015 13:28, Michael Niedermayer wrote: > > On Wed, Dec 16, 2015 at 08:20:18PM +0100, Andreas Cadhalpun wrote: > >> More don't fit into the integer output. > >> > >> Also use get_bits_long, since get_bits only supports

Re: [FFmpeg-devel] Vertical bands with RT21 (Intel Real Time Video 2.1) codec

2015-12-17 Thread Mats Peterson
On 12/18/2015 04:13 AM, Mats Peterson wrote: Samples: https://drive.google.com/open?id=0B3_pEBoLs0faei1Sd1h6SG11QlE Mats The clips are sample clips from Video for Windows, in case anyone wonders. I haven't created them myself. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/

Re: [FFmpeg-devel] Vertical bands with RT21 (Intel Real Time Video 2.1) codec

2015-12-17 Thread Mats Peterson
On 12/18/2015 05:12 AM, Mats Peterson wrote: On 12/18/2015 04:13 AM, Mats Peterson wrote: Samples: https://drive.google.com/open?id=0B3_pEBoLs0faei1Sd1h6SG11QlE Mats The clips are sample clips from Video for Windows, in case anyone wonders. I haven't created them myself. Mats The output