[FFmpeg-devel] [PATCH] avcodec: Use get_ue_golomb_long() when needed

2015-12-28 Thread Mark Harris
get_ue_golomb() cannot decode values larger than 8190 (the maximum value that can be golomb encoded in 25 bits) and produces the error "Invalid UE golomb code" if a larger value is encountered. Use get_ue_golomb_long() instead (which supports 63 bits, up to 4294967294) when valid h264/hevc values

[FFmpeg-devel] [PATCH v2] lavf/qtpalette: Treat 1-bit video as palettized

2015-12-28 Thread Mats Peterson
I forgot some stuff. Patch description follows: This patch for qtpalette.c and qtpalette.h adds 1-bit video to the "palettized video" category, since if the video sample description contains a palette, the two colors in the palette can be any color, not necessarily black & white. Unfortunately,

Re: [FFmpeg-devel] [PATCH] lavfi/af_anequalizer: remove cabs, cexp dependencies

2015-12-28 Thread Matt Oliver
On 29 December 2015 at 11:51, Ganesh Ajjanagadde wrote: > On Mon, Dec 28, 2015 at 8:22 AM, Paul B Mahol wrote: > > On 12/27/15, Ganesh Ajjanagadde wrote: > >> Replaces by real arithmetic. Tested the validity of these > transformations > >> separately. > >> > >> Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] avformat/concatdec: get bit_rate from internal format

2015-12-28 Thread Zhang Rui
--- libavformat/concatdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/concatdec.c b/libavformat/concatdec.c index d21805f..4c7217c 100644 --- a/libavformat/concatdec.c +++ b/libavformat/concatdec.c @@ -310,6 +310,7 @@ static int open_file(AVFormatContext *avf, unsigned fileno

[FFmpeg-devel] [PATCH] lavf/qtpalette: Treat 1-bit video as palettized

2015-12-28 Thread Mats Peterson
This patch for qtpalette.c and qtpalette.h adds 1-bit video to the "palettized video" category, since if the video sample description contains a palette, the two colors in the palette can be any color, not necessarily black & white. Unfortunately, I've noticed that the qtrle (QuickTime Animation)

Re: [FFmpeg-devel] Doxy documentation for qtpalette.c vanished?

2015-12-28 Thread Mats Peterson
On 12/29/2015 05:26 AM, Mats Peterson wrote: Excuse me, but where did the doxy documentation for ff_get_qtpalette() in qtpalette.c go? Michael? Mats Ah, it's in qtpalette.h. Sorry. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ ___ ffmpeg-

Re: [FFmpeg-devel] Doxy documentation for qtpalette.c vanished?

2015-12-28 Thread Stephan Holljes
On Tue, Dec 29, 2015 at 5:26 AM, Mats Peterson wrote: > Excuse me, but where did the doxy documentation for ff_get_qtpalette() in > qtpalette.c go? Michael? > > Mats > > -- > Mats Peterson > http://matsp888.no-ip.org/~mats/ > ___ > ffmpeg-devel mailing l

[FFmpeg-devel] Doxy documentation for qtpalette.c vanished?

2015-12-28 Thread Mats Peterson
Excuse me, but where did the doxy documentation for ff_get_qtpalette() in qtpalette.c go? Michael? Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-d

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec.c: Copy QuickTime palette to st->codec->extradata

2015-12-28 Thread Mats Peterson
On 12/29/2015 03:23 AM, Mats Peterson wrote: In case this patch got lost in my endless stream of previous posts, I'll repost it. Description of the patch follows: In order to make MPlayer use the correct palette when playing palettized QuickTime video in Matroska, the palette will need to be cop

Re: [FFmpeg-devel] [PATCH] avcodec/srtdec: Keep exact end times

2015-12-28 Thread Rodger Combs
> On Dec 3, 2015, at 03:30, Eelco Lempsink wrote: > > When converting SRT to SRT (to normalize) or WebVTT the end timestamps were > modified compared to the original. > > Fixes trac 4783. > > NOTE: The FATE test 'sub-srt' fails after this patch, because the end times of > the Dialogue lines ch

[FFmpeg-devel] [PATCH] lavf/matroskadec.c: Copy QuickTime palette to st->codec->extradata

2015-12-28 Thread Mats Peterson
In case this patch got lost in my endless stream of previous posts, I'll repost it. Description of the patch follows: In order to make MPlayer use the correct palette when playing palettized QuickTime video in Matroska, the palette will need to be copied to st->codec->extradata in matroskadec.c.

Re: [FFmpeg-devel] [RFC 1/3] daaladec: Implement a native Daala decoder

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 04:58:48PM +, Rostislav Pehlivanov wrote: > This commit is a work in progress Daala decoder implemented in FFmpeg. > Comments are requested. > > The current decoder is able to decode lossless Daala encoded files (the > output of this decoder is still identical to the ou

Re: [FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-28 Thread Ganesh Ajjanagadde
On Mon, Dec 28, 2015 at 8:00 AM, Ganesh Ajjanagadde wrote: > On Mon, Dec 28, 2015 at 12:14 AM, Paul B Mahol wrote: >> On 12/28/15, Ganesh Ajjanagadde wrote: >>> Signed-off-by: Ganesh Ajjanagadde >>> --- >>> libavfilter/af_anequalizer.c | 4 ++-- >>> 1 file changed, 2 insertions(+), 2 deletions

Re: [FFmpeg-devel] [PATCH] lavfi/af_anequalizer: remove cabs, cexp dependencies

2015-12-28 Thread Ganesh Ajjanagadde
On Mon, Dec 28, 2015 at 8:22 AM, Paul B Mahol wrote: > On 12/27/15, Ganesh Ajjanagadde wrote: >> Replaces by real arithmetic. Tested the validity of these transformations >> separately. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> configure| 1 - >> libavfilter/af_aneq

[FFmpeg-devel] [PATCH] avfilter/avf_showspectrum: properly calculate w factor

2015-12-28 Thread Paul B Mahol
It is used in calculating loudness of each frequency bin. This increases contrast for all scale modes except log. Signed-off-by: Paul B Mahol --- libavfilter/avf_showspectrum.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavfilter/avf_showspectrum.c b/libavfilter/

Re: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-28 Thread Marton Balint
On Mon, 28 Dec 2015, Michael Niedermayer wrote: On Sun, Dec 27, 2015 at 11:09:42PM +0100, Balint Marton wrote: Hi Michael, The patch you committed seems to break the cropping to even width / height as required by SDL overlay code. Also there can be a use case for inserting the auto rotati

Re: [FFmpeg-devel] support for reading / writing encrypted MP4 files

2015-12-28 Thread Eran Kornblau
> > +case MKTAG('e','n','c','v'):// encrypted video > > +case MKTAG('e','n','c','a'):// encrypted audio > > +id = mov_codec_id(st, format); > > +st->codec->codec_id = id; > > this seems missing a check for st->codec->codec_id being "unset" > before setting i

[FFmpeg-devel] [PATCH] build: make out-of-tree builds bit-identical to in-tree builds

2015-12-28 Thread Andreas Cadhalpun
Previously the full source path was embedded inconsistently in the debug information between in-tree/out-of-tree builds. The 'vpath %.inc' becomes necessary for finding libavfilter/all_channel_layouts.inc in out-of-tree builds. The full source path is still embedded in the debug information, but

Re: [FFmpeg-devel] [PATCH] doc: make apidoc output independent of SRC_PATH

2015-12-28 Thread Andreas Cadhalpun
On 20.12.2015 23:31, Andreas Cadhalpun wrote: > Previously it included the SRC_PATH in every title. > > Signed-off-by: Andreas Cadhalpun > --- > doc/Makefile| 7 --- > doc/doxy-wrapper.sh | 11 ++- > 2 files changed, 10 insertions(+), 8 deletions(-) > > diff --git a/doc/Mak

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:58 PM, Ganesh Ajjanagadde wrote: @Mats: this is a good example for why it is very helpful to reduce the volume of email messages, and to think over them before sending to avoid small additions/removals/clarifications. Yes, i can fully understand that. I'm going to behave in t

Re: [FFmpeg-devel] support for reading / writing encrypted MP4 files

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 05:29:35AM +, Eran Kornblau wrote: > Bumping up this thread (latest patch attached) > > Happy holidays ! happy holidays as well! > > Eran > [...] > @@ -4007,6 +4009,162 @@ static int mov_read_free(MOVContext *c, AVIOContext > *pb, MOVAtom atom) > return 0; >

Re: [FFmpeg-devel] [PATCH 2/2] x86/vf_stereo3d: make ff_anaglyph_sse4 work on x86_32

2015-12-28 Thread James Almer
On 12/28/2015 5:15 AM, Paul B Mahol wrote: > On 12/27/15, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavfilter/x86/vf_stereo3d.asm| 47 >> +++--- >> libavfilter/x86/vf_stereo3d_init.c | 2 +- >> 2 files changed, 45 insertions(+), 4 deletions

Re: [FFmpeg-devel] [RFC 1/3] daaladec: Implement a native Daala decoder

2015-12-28 Thread Carl Eugen Hoyos
Rostislav Pehlivanov gmail.com> writes: > The current decoder is able to decode lossless Daala > encoded files (the output of this decoder is still > identical to the output) to the input? > Most of the code here was written using libdaala as > a reference, after optimizing, refactoring and

[FFmpeg-devel] [RFC 2/3] daala_parser: add a Daala parser

2015-12-28 Thread Rostislav Pehlivanov
Not much to say about this other than "it works". Signed-off-by: Rostislav Pehlivanov --- configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c| 1 + libavcodec/daala_parser.c | 50 +++ 4 files changed, 53 inse

[FFmpeg-devel] [RFC 3/3] oggenc: add support for remuxing Daala video into OGG container

2015-12-28 Thread Rostislav Pehlivanov
Signed-off-by: Rostislav Pehlivanov --- libavformat/oggenc.c | 27 +-- 1 file changed, 17 insertions(+), 10 deletions(-) diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c index 4907512..87dcdde 100644 --- a/libavformat/oggenc.c +++ b/libavformat/oggenc.c @@ -452,7

Re: [FFmpeg-devel] [PATCH] lavfi/af_anequalizer: remove cabs, cexp dependencies

2015-12-28 Thread Paul B Mahol
On 12/27/15, Ganesh Ajjanagadde wrote: > Replaces by real arithmetic. Tested the validity of these transformations > separately. > > Signed-off-by: Ganesh Ajjanagadde > --- > configure| 1 - > libavfilter/af_anequalizer.c | 25 + > 2 files changed, 17

Re: [FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-28 Thread Ganesh Ajjanagadde
On Mon, Dec 28, 2015 at 12:14 AM, Paul B Mahol wrote: > On 12/28/15, Ganesh Ajjanagadde wrote: >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavfilter/af_anequalizer.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/libavfilter/af_anequalizer.c b/libavfilter/a

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Ganesh Ajjanagadde
On Mon, Dec 28, 2015 at 6:39 AM, Nicolas George wrote: > L'octidi 8 nivôse, an CCXXIV, Michael Niedermayer a écrit : >> for reference: (similar code prior to the patches) > > Indeed. Mats already pointed one of them. This would warrant some > refactoring. At the very least: > > ff_frame_new_pa

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:41 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Well, I guess I have to temper myself a bit. But I don't accept that someone else grabs the honor for something I've written. Would you, once again? Can you understand my point whatsoever? I hav

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > Well, I guess I have to temper myself a bit. But I don't accept that someone > else grabs the honor for something I've written. Would you, once again? Can > you understand my point whatsoever? I have seen your point. I had already seen your p

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:33 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : So it's OK for him to say he wrote the significant parts of the patch? It's ok for him to say that, it's ok for other people to disagree and prove their point. It's not ok to rehash the questio

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:27 PM, Mats Peterson wrote: On 12/28/2015 04:26 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Listen, Nicolas. I'm not DENOUNCING Carl Eugen by saying he didn't write the significant parts of the patch. Where did you get that from? Would YOU li

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > So it's OK for him to say he wrote the significant parts of the patch? It's ok for him to say that, it's ok for other people to disagree and prove their point. It's not ok to rehash the question endlessly. It's not ok to dig up irrelevant h

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:26 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Listen, Nicolas. I'm not DENOUNCING Carl Eugen by saying he didn't write the significant parts of the patch. Where did you get that from? Would YOU like someone else grabbing the honor for somethi

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > Listen, Nicolas. I'm not DENOUNCING Carl Eugen by saying he didn't write the > significant parts of the patch. Where did you get that from? Would YOU like > someone else grabbing the honor for something you've written? I don't think > you woul

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:04 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : The point is to make people understand he didn't write "significant parts" of the latest patch. I want to make that clear. Not that I care who wrote it, as long as it works. I just don't like his

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:07 PM, Mats Peterson wrote: On 12/28/2015 04:04 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : The point is to make people understand he didn't write "significant parts" of the latest patch. I want to make that clear. Not that I care who wrote i

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 04:04 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : The point is to make people understand he didn't write "significant parts" of the latest patch. I want to make that clear. Not that I care who wrote it, as long as it works. I just don't like his

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > The point is to make people understand he didn't write "significant parts" > of the latest patch. I want to make that clear. Not that I care who wrote > it, as long as it works. I just don't like his grabbing the honor for > something he didn'

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 03:48 PM, Mats Peterson wrote: On 12/28/2015 03:45 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : By the way, here's a clip from the old patch (THIS one had significant parts written by Carl alright) that got rave reviews from Hendrik Leppkes, call

Re: [FFmpeg-devel] [PATCH] avformat/http: Documented http_proxy option

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 09:50:15AM +, Joel Holdsworth wrote: > --- > doc/protocols.texi | 3 +++ > 1 file changed, 3 insertions(+) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The bravest are surely those who have the clearest vision of w

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 03:45 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : By the way, here's a clip from the old patch (THIS one had significant parts written by Carl alright) that got rave reviews from Hendrik Leppkes, calling it "an extremely ugly hack", complaining a

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 03:39 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Michael Niedermayer a écrit : for reference: (similar code prior to the patches) Indeed. Mats already pointed one of them. This would warrant some refactoring. At the very least: ff_frame_new_palette(frame, palet

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > By the way, here's a clip from the old patch (THIS one had significant parts > written by Carl alright) that got rave reviews from Hendrik Leppkes, calling > it "an extremely ugly hack", complaining at the calling into another demuxer > (by us

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 03:32 PM, Mats Peterson wrote: On 12/28/2015 03:29 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 12:07:38PM +0100, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Michael, he's talking about the OLD patch that was never applied. My patch has b

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Michael Niedermayer a écrit : > for reference: (similar code prior to the patches) Indeed. Mats already pointed one of them. This would warrant some refactoring. At the very least: ff_frame_new_palette(frame, palette); Then Ganesh will be able to add av_warn_unu

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 03:29 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 12:07:38PM +0100, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Michael, he's talking about the OLD patch that was never applied. My patch has been written from scratch, more or less. I did

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 12:07:38PM +0100, Nicolas George wrote: > L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > > Michael, he's talking about the OLD patch that was never applied. My patch > > has been written from scratch, more or less. I did borrowed some palette > > loops from mov.c, b

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:45 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : I'm afraid he won't agree on ANYTHING I say, after my initial treatment of him on the bug tracker. I have excused myself for that as well, but he's slow to forget... Maybe if you stopped ADDING

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:39 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 02:18:14PM +0100, Mats Peterson wrote: On 12/28/2015 02:16 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 02:09:54PM +0100, Mats Peterson wrote: On 12/28/2015 02:03 PM, Michael Niedermayer wrote: On Mon, Dec 28

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > I'm afraid he won't agree on ANYTHING I say, after my initial treatment of > him on the bug tracker. I have excused myself for that as well, but he's > slow to forget... Maybe if you stopped ADDING insults like that, it would start making thi

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:39 PM, Michael Niedermayer wrote: 1AM +0100, Mats Peterson wrote: I'm afraid he won't agree on ANYTHING I say, after my initial treatment of him on the bug tracker. I have excused myself for that as well, but he's slow to forget... Mats -- Mats Peterson http://matsp888.no-

Re: [FFmpeg-devel] [PATCH] lavf/matroskadec.c: Copy QuickTime palette to extradata

2015-12-28 Thread wm4
On Mon, 28 Dec 2015 05:18:41 +0100 Mats Peterson wrote: > I still insist on copying the QuickTime palette to extradata in > matroskadec.c, since it's currently needed for MPlayer to use the > correct palette. As I have explained so many times before, MPlayer, for > some inexplicable reason, curre

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 02:18:14PM +0100, Mats Peterson wrote: > On 12/28/2015 02:16 PM, Michael Niedermayer wrote: > >On Mon, Dec 28, 2015 at 02:09:54PM +0100, Mats Peterson wrote: > >>On 12/28/2015 02:03 PM, Michael Niedermayer wrote: > >>>On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson w

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:18 PM, Mats Peterson wrote: On 12/28/2015 02:16 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 02:09:54PM +0100, Mats Peterson wrote: On 12/28/2015 02:03 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: On 12/28/2015 10:34

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:16 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 02:09:54PM +0100, Mats Peterson wrote: On 12/28/2015 02:03 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: On 12/28/2015 10:34 AM, Michael Niedermayer wrote: On Mon, Dec 28

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 02:09:54PM +0100, Mats Peterson wrote: > On 12/28/2015 02:03 PM, Michael Niedermayer wrote: > >On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: > >>On 12/28/2015 10:34 AM, Michael Niedermayer wrote: > >>>On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyo

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:09 PM, Mats Peterson wrote: On 12/28/2015 02:03 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: On 12/28/2015 10:34 AM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: Michael Niedermay

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 02:03 PM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: On 12/28/2015 10:34 AM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: Michael Niedermayer niedermayer.cc> writes: Patch splited in

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 11:46:21AM +0100, Mats Peterson wrote: > On 12/28/2015 10:34 AM, Michael Niedermayer wrote: > >On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: > >>Michael Niedermayer niedermayer.cc> writes: > >> > >>>Patch splited in move and matroska part > >>>i removed

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 12:25 PM, Mats Peterson wrote: On 12/28/2015 12:21 PM, Mats Peterson wrote: On 12/28/2015 12:07 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Michael, he's talking about the OLD patch that was never applied. My patch has been written from scratch

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 12:21 PM, Mats Peterson wrote: On 12/28/2015 12:07 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Michael, he's talking about the OLD patch that was never applied. My patch has been written from scratch, more or less. I did borrowed some palette lo

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 12:07 PM, Nicolas George wrote: L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : Michael, he's talking about the OLD patch that was never applied. My patch has been written from scratch, more or less. I did borrowed some palette loops from mov.c, but I have also attributed the

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Nicolas George
L'octidi 8 nivôse, an CCXXIV, Mats Peterson a écrit : > Michael, he's talking about the OLD patch that was never applied. My patch > has been written from scratch, more or less. I did borrowed some palette > loops from mov.c, but I have also attributed the previous authors at the top > of qtpalette

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 11:46 AM, Mats Peterson wrote: On 12/28/2015 10:34 AM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: Michael Niedermayer niedermayer.cc> writes: Patch splited in move and matroska part i removed this memcpy() for now from what i co

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 10:34 AM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: Michael Niedermayer niedermayer.cc> writes: Patch splited in move and matroska part i removed this memcpy() for now from what i commited as there is clearly no consenus on it

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Mats Peterson
On 12/28/2015 10:34 AM, Michael Niedermayer wrote: On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: Michael Niedermayer niedermayer.cc> writes: Patch splited in move and matroska part i removed this memcpy() for now from what i commited as there is clearly no consenus on it

[FFmpeg-devel] [PATCH] avformat/http: Documented http_proxy option

2015-12-28 Thread Joel Holdsworth
--- doc/protocols.texi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/protocols.texi b/doc/protocols.texi index 7c6b9d4..8b2e310 100644 --- a/doc/protocols.texi +++ b/doc/protocols.texi @@ -240,6 +240,9 @@ If set to 1 use chunked Transfer-Encoding for posts, default is 1. @item conte

Re: [FFmpeg-devel] [PATCH v9] lavf: palettized QuickTime video in Matroska

2015-12-28 Thread Michael Niedermayer
On Mon, Dec 28, 2015 at 03:10:18AM +, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > > > Patch splited in move and matroska part > > i removed this memcpy() for now from what i > > commited as there is clearly no consenus on it > > I would really have appreciated a

Re: [FFmpeg-devel] [PATCH] avcodec/utils: fix check for invalid timebase when decoding subtitles

2015-12-28 Thread Hendrik Leppkes
On Sun, Dec 27, 2015 at 9:29 PM, Andreas Cadhalpun wrote: > On 27.12.2015 21:13, Hendrik Leppkes wrote: >> On Sun, Dec 27, 2015 at 9:03 PM, Andreas Cadhalpun >> wrote: >>> On 27.12.2015 20:43, Hendrik Leppkes wrote: On Sun, Dec 27, 2015 at 8:29 PM, Andreas Cadhalpun wrote: > On 27.

Re: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-28 Thread Michael Niedermayer
On Sun, Dec 27, 2015 at 11:09:42PM +0100, Balint Marton wrote: > Hi Michael, > > The patch you committed seems to break the cropping to even width / > height as required by SDL overlay code. > > Also there can be a use case for inserting the auto rotation filter > after the user provided filter

Re: [FFmpeg-devel] ffplay: insertion point of the auto rotation filter - Github ticket #141

2015-12-28 Thread Michael Niedermayer
On Sun, Dec 27, 2015 at 11:09:42PM +0100, Balint Marton wrote: > Hi Michael, > > The patch you committed seems to break the cropping to even width / > height as required by SDL overlay code. > > Also there can be a use case for inserting the auto rotation filter > after the user provided filter c

Re: [FFmpeg-devel] [PATCH 2/2] x86/vf_stereo3d: make ff_anaglyph_sse4 work on x86_32

2015-12-28 Thread Paul B Mahol
On 12/27/15, James Almer wrote: > Signed-off-by: James Almer > --- > libavfilter/x86/vf_stereo3d.asm| 47 > +++--- > libavfilter/x86/vf_stereo3d_init.c | 2 +- > 2 files changed, 45 insertions(+), 4 deletions(-) > > diff --git a/libavfilter/x86/vf_stereo3d.as

Re: [FFmpeg-devel] [PATCH 2/2] lavfi/af_anequalizer: replace pow(x, -2) by 1/(x*x)

2015-12-28 Thread Paul B Mahol
On 12/28/15, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavfilter/af_anequalizer.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavfilter/af_anequalizer.c b/libavfilter/af_anequalizer.c > index d7b5b6c..649c0b9 100644 > --- a/libavfi