Re: [FFmpeg-devel] [PATCH] filtfmts: Avoid null pointer dereferences

2015-04-03 Thread Carl Eugen Hoyos
Himangi Saraogi gmail.com> writes: { \ > -if (filter_ctx->inout##puts[i]->type == AVMEDIA_TYPE_VIDEO) { \ > +if (filter_ctx->inout##puts[i]) {\ > +if (filter_ctx->inout##puts[i]->type == AVMEDIA_TYPE_VIDEO) { \ Please resen

Re: [FFmpeg-devel] [PATCH]Support alaw in imkh files

2015-04-03 Thread Carl Eugen Hoyos
On Thursday 02 April 2015 11:47:08 am Carl Eugen Hoyos wrote: > > +} else if (m->imkh_cctv && startcode == 0x1c0) { > > +codec_id = AV_CODEC_ID_PCM_ALAW; > > Maybe with "request_probe=50;" - the manual of one > of the camera claims that MP2 is a possible audio > codec. New patc

[FFmpeg-devel] [PATCH] filtfmts: Avoid null pointer dereferences

2015-04-03 Thread Himangi Saraogi
--- libavfilter/filtfmts.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavfilter/filtfmts.c b/libavfilter/filtfmts.c index c1025b9..3e2a011 100644 --- a/libavfilter/filtfmts.c +++ b/libavfilter/filtfmts.c @@ -34,6 +34,7 @@ static void print_formats(AVFilterContext *filter_ctx) #

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 03:54:08AM +0200, Mariusz Szczepańczyk wrote: > --- > doc/APIchanges| 5 + > libavformat/avio.h| 5 - > libavformat/version.h | 2 +- > 3 files changed, 10 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B1

Re: [FFmpeg-devel] [PATCH 2/3] doc/examples: Handle new types in avio_list_dir

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 03:54:09AM +0200, Mariusz Szczepańczyk wrote: > --- > doc/examples/avio_list_dir.c | 6 ++ > 1 file changed, 6 insertions(+) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into democracies and democ

Re: [FFmpeg-devel] [PATCH] filtfmts: Avoid null pointer dereferences

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 02:25:13PM +0530, Himangi Saraogi wrote: > --- > libavfilter/filtfmts.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/libavfilter/filtfmts.c b/libavfilter/filtfmts.c > index c1025b9..3e2a011 100644 > --- a/libavfilter/filtfmts.c > +++ b/libavfilter/filtfmt

Re: [FFmpeg-devel] [PATCH]Support alaw in imkh files

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 10:45:53AM +0200, Carl Eugen Hoyos wrote: > On Thursday 02 April 2015 11:47:08 am Carl Eugen Hoyos wrote: > > > +} else if (m->imkh_cctv && startcode == 0x1c0) { > > > +codec_id = AV_CODEC_ID_PCM_ALAW; > > > > Maybe with "request_probe=50;" - the manual o

Re: [FFmpeg-devel] [PATCH] Relaxed pattern to find ProRes in MXF.

2015-04-03 Thread Steve Dierker
Hello, > > +{ { > 0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x02,0x0d,0x01,0x03,0x01,0x02,0x04,0x > 60,0x02 > }, 14, AV_CODEC_ID_MPEG2VIDEO }, /* Ikegami */ > > Errm where did this coming from, its unrelated (and wrong afaik as > its a container UL). > Oh, sorry this is unrelated, I'm current

Re: [FFmpeg-devel] [PATCH] Relaxed pattern to find ProRes in MXF.

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 12:55:15PM +0200, Steve Dierker wrote: > Hello, > > > > > +{ { > > 0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x02,0x0d,0x01,0x03,0x01,0x02,0x04,0x > > 60,0x02 > > }, 14, AV_CODEC_ID_MPEG2VIDEO }, /* Ikegami */ > > > > Errm where did this coming from, its unrelated (and wrong

[FFmpeg-devel] [PATCH]Fix fps diplay for very small fps

2015-04-03 Thread Carl Eugen Hoyos
Hi! Attached patch improves "ffmpeg -i" output for the sample from ticket #4393. Please comment, Carl Eugen diff --git a/libavformat/dump.c b/libavformat/dump.c index 56b37ff..9a7035c 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -118,7 +118,9 @@ void av_pkt_dump_log2(void *avcl, i

[FFmpeg-devel] Patch: fix bug for POWER8 LE in file avcodec/ppc/fdctdsp.c

2015-04-03 Thread rongyan
Hi, The attachment please find the patch to fix bug on POWER8 LE for jpeg picture convert. The fucntion ff_fdct_altivec() is fixed and the macros VEC_FMERGEH and VEC_FMERGEL are added. The fate test results is here: Thanks for your review. Rong Yan 0001-avcodec-ppc-fdctdsp.c-POWER

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Nicolas George
Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : > diff --git a/libavformat/avio.h b/libavformat/avio.h > index f61a8f8..51913e3 100644 > --- a/libavformat/avio.h > +++ b/libavformat/avio.h > @@ -63,7 +63,10 @@ enum AVIODirEntryType { > AVIO_ENTRY_NAMED_PIPE, > AVIO_ENT

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Lukasz Marek
On 3 April 2015 at 15:22, Nicolas George wrote: > Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : > > diff --git a/libavformat/avio.h b/libavformat/avio.h > > index f61a8f8..51913e3 100644 > > --- a/libavformat/avio.h > > +++ b/libavformat/avio.h > > @@ -63,7 +63,10 @@ enum AV

[FFmpeg-devel] [PATCH] matroskadec: export cover art correctly

2015-04-03 Thread wm4
Generally, libavformat exports cover art pictures as video streams with 1 packet and AV_DISPOSITION_ATTACHED_PIC set. Only matroskadec exported it as attachment with codec_id set to AV_CODEC_ID_MJPEG. Obviously, this should be consistent, so change the Matroska demuxer to export a AV_DISPOSITION_A

[FFmpeg-devel] [PATCH] filtfmts: Avoid null pointer dereference

2015-04-03 Thread Himangi Saraogi
--- libavfilter/filtfmts.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/libavfilter/filtfmts.c b/libavfilter/filtfmts.c index c1025b9..46a2d94 100644 --- a/libavfilter/filtfmts.c +++ b/libavfilter/filtfmts.c @@ -75,6 +75,7 @@ int main(int argc, char **argv)

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 03:22:29PM +0200, Nicolas George wrote: > Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : > > diff --git a/libavformat/avio.h b/libavformat/avio.h > > index f61a8f8..51913e3 100644 > > --- a/libavformat/avio.h > > +++ b/libavformat/avio.h > > @@ -63,7 +63

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Lukasz Marek
On 3 April 2015 at 16:05, Lukasz Marek wrote: > On 3 April 2015 at 15:22, Nicolas George wrote: > >> Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : >> > diff --git a/libavformat/avio.h b/libavformat/avio.h >> > index f61a8f8..51913e3 100644 >> > --- a/libavformat/avio.h >> >

Re: [FFmpeg-devel] [PATCH]Fix fps diplay for very small fps

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 02:14:34PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch improves "ffmpeg -i" output for the sample from > ticket #4393. > > Please comment, Carl Eugen > dump.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > cb560fbeeb217782f01b5e94a5497b23589

Re: [FFmpeg-devel] [PATCH] filtfmts: Avoid null pointer dereference

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 07:46:05PM +0530, Himangi Saraogi wrote: > --- > libavfilter/filtfmts.c | 14 +- > 1 file changed, 13 insertions(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Good people do not need laws t

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: last frame duplication heuristic

2015-04-03 Thread Carl Eugen Hoyos
Michael Niedermayer gmx.at> writes: > This improves the last frames duration with CFR and > when the input durations are inaccurate or missing Ping. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Mariusz Szczepańczyk
On Fri, Apr 3, 2015 at 3:22 PM, Nicolas George wrote: > Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : > > diff --git a/libavformat/avio.h b/libavformat/avio.h > > index f61a8f8..51913e3 100644 > > --- a/libavformat/avio.h > > +++ b/libavformat/avio.h > > @@ -63,7 +63,10 @@ e

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Mariusz Szczepańczyk
On Fri, Apr 3, 2015 at 4:52 PM, Lukasz Marek wrote: > On 3 April 2015 at 16:05, Lukasz Marek wrote: > > > On 3 April 2015 at 15:22, Nicolas George wrote: > > > >> Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : > >> > diff --git a/libavformat/avio.h b/libavformat/avio.h > >>

Re: [FFmpeg-devel] Patch: fix bug for POWER8 LE in file avcodec/ppc/fdctdsp.c

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 09:14:35PM +0800, rongyan wrote: > Hi, > The attachment please find the patch to fix bug on POWER8 LE for jpeg > picture convert. The fucntion ff_fdct_altivec() is fixed and the macros > VEC_FMERGEH and VEC_FMERGEL are added. > > The fate test results is here: > >

[FFmpeg-devel] [PATCH] avcodec/nvenc: Add options for level and tier

2015-04-03 Thread Timo Rothenpieler
--- libavcodec/nvenc.c | 111 - 1 file changed, 110 insertions(+), 1 deletion(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index f77a982..d4e0a39 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -129,6 +129,12 @@ typedef st

Re: [FFmpeg-devel] [PATCH] libavcodec/exr.c : exr lossy compression B44 based on OpenEXR code

2015-04-03 Thread Paul B Mahol
On 3/27/15, greeshma wrote: > The corrected patch is attached. > PFA > > Greeshma > It is missing break. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] chorus filter

2015-04-03 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/af_chorus.c | 380 +++ libavfilter/allfilters.c | 1 + 3 files changed, 382 insertions(+) create mode 100644 libavfilter/af_chorus.c diff --git a/libavfilter/Makefile b/l

[FFmpeg-devel] [PATCH 2/4] mpeg4: only show the divx_packed warning, when a packed frame is present

2015-04-03 Thread Andreas Cadhalpun
Otherwise it is shown even after the file was fixed with avidemux, because the userdata string still ends with 'p'. Signed-off-by: Andreas Cadhalpun --- libavcodec/mpeg4videodec.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/mpeg4videodec.c b/libavc

[FFmpeg-devel] [PATCH 1/4] mpeg4: use a self-explanatory #define MAX_NVOP_SIZE

2015-04-03 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavcodec/h263dec.c| 2 +- libavcodec/mpeg4video.h | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c index a1981a3..0dde897 100644 --- a/libavcodec/h263dec.c +++ b/libavcodec/h263dec.c @@ -4

[FFmpeg-devel] [PATCH 4/4] mpeg4: suggest mpeg4_unpack_bframes bsf instead of avidemux/VirtualDub

2015-04-03 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavcodec/mpeg4videodec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index 80dfb91..b8fd50b 100644 --- a/libavcodec/mpeg4videodec.c +++ b/libavcodec/mpeg4videodec.c @@ -2664,7 +2

[FFmpeg-devel] [PATCH 3/4] avcodec: add unpack packed B-frames bitstream filter

2015-04-03 Thread Andreas Cadhalpun
Fixes Ticket #2913 Signed-off-by: Andreas Cadhalpun --- Changelog | 1 + doc/bitstream_filters.texi| 20 libavcodec/Makefile | 1 + libavcodec/allcodecs.c| 1 + libavcodec/mpeg4_unpack_bframes_bsf.c | 200 +

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Nicolas George
Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : > I think there should be a separate function to construct a new url from a > url > and an entry object, that covers these cases or calls > av_append_path_component() for regular entries. Or we could just skip 20 years of API desi

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: last frame duplication heuristic

2015-04-03 Thread Michael Niedermayer
On Wed, Nov 19, 2014 at 01:18:41PM +0100, Michael Niedermayer wrote: > This improves the last frames duration with CFR and when the input durations > are inaccurate or missing > > Signed-off-by: Michael Niedermayer applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: fix indention

2015-04-03 Thread Michael Niedermayer
On Thu, Nov 20, 2014 at 10:52:42AM +0100, Clément Bœsch wrote: > indentation* changed applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Freedom in capitalist society always remains about the same as it was in ancient Greek republics: Freedom for s

Re: [FFmpeg-devel] [PATCH 3/3] lavf/libsmbclient: implement directory listing callbacks

2015-04-03 Thread Lukasz Marek
On 3 April 2015 at 03:54, Mariusz Szczepańczyk wrote: > --- > libavformat/libsmbclient.c | 108 > + > 1 file changed, 108 insertions(+) > LGTM, I don't push yet as there is some discussion on other thread. I will be unavailable during following few da

Re: [FFmpeg-devel] [PATCH 1/4] mpeg4: use a self-explanatory #define MAX_NVOP_SIZE

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 07:14:16PM +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/h263dec.c| 2 +- > libavcodec/mpeg4video.h | 3 +++ > 2 files changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c > index

Re: [FFmpeg-devel] [PATCH 2/4] mpeg4: only show the divx_packed warning, when a packed frame is present

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 07:14:31PM +0200, Andreas Cadhalpun wrote: > Otherwise it is shown even after the file was fixed with avidemux, > because the userdata string still ends with 'p'. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpeg4videodec.c | 12 ++-- > 1 file changed,

Re: [FFmpeg-devel] [PATCH 1/4] mpeg4: use a self-explanatory #define MAX_NVOP_SIZE

2015-04-03 Thread Andreas Cadhalpun
On 03.04.2015 20:20, Michael Niedermayer wrote: > On Fri, Apr 03, 2015 at 07:14:16PM +0200, Andreas Cadhalpun wrote: >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/h263dec.c| 2 +- >> libavcodec/mpeg4video.h | 3 +++ >> 2 files changed, 4 insertions(+), 1 deletion(-) >> >> diff --gi

[FFmpeg-devel] [PATCH] examples/avio_list_dir: init/deinit network

2015-04-03 Thread Lukasz Marek
Signed-off-by: Lukasz Marek --- doc/examples/avio_list_dir.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/doc/examples/avio_list_dir.c b/doc/examples/avio_list_dir.c index 281b5af..4060ba6 100644 --- a/doc/examples/avio_list_dir.c +++ b/doc/examples/avio_list_dir.c

Re: [FFmpeg-devel] [PATCH 1/3] lavf/avio: Add new types to AVIODirEntryType, bump minor version

2015-04-03 Thread Lukasz Marek
On 03.04.2015 16:19, Michael Niedermayer wrote: On Fri, Apr 03, 2015 at 03:22:29PM +0200, Nicolas George wrote: Le quartidi 14 germinal, an CCXXIII, Mariusz Szczepańczyk a écrit : diff --git a/libavformat/avio.h b/libavformat/avio.h index f61a8f8..51913e3 100644 --- a/libavformat/avio.h +++ b/l

Re: [FFmpeg-devel] [PATCH 1/4] mpeg4: use a self-explanatory #define MAX_NVOP_SIZE

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 08:37:29PM +0200, Andreas Cadhalpun wrote: > On 03.04.2015 20:20, Michael Niedermayer wrote: > > On Fri, Apr 03, 2015 at 07:14:16PM +0200, Andreas Cadhalpun wrote: > >> Signed-off-by: Andreas Cadhalpun > >> --- > >> libavcodec/h263dec.c| 2 +- > >> libavcodec/mpeg4vide

Re: [FFmpeg-devel] [PATCH] matroskadec: export cover art correctly

2015-04-03 Thread Michael Niedermayer
On Fri, Apr 03, 2015 at 04:11:53PM +0200, wm4 wrote: > Generally, libavformat exports cover art pictures as video streams with > 1 packet and AV_DISPOSITION_ATTACHED_PIC set. Only matroskadec exported > it as attachment with codec_id set to AV_CODEC_ID_MJPEG. > > Obviously, this should be consiste

Re: [FFmpeg-devel] Patch for High color and High bit-depth support

2015-04-03 Thread Debargha Mukherjee
On Thu, Apr 2, 2015 at 3:10 PM, James Zern wrote: > On Thu, Apr 2, 2015 at 12:19 PM, Debargha Mukherjee > wrote: > > [...] > > > > Updated patch to handle the checks better. > > > > > +#if CONFIG_LIBVPX_VP9_ENCODER > > +static int set_pix_fmt(AVCodecContext *avctx, vpx_codec_caps_t > codec_caps,

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: Add options for level and tier

2015-04-03 Thread Philip Langdale
On Fri, 3 Apr 2015 18:23:05 +0200 Timo Rothenpieler wrote: > --- > libavcodec/nvenc.c | 111 > - 1 file changed, > 110 insertions(+), 1 deletion(-) Needs a changelog description but otherwise looks fine. --phil ___

[FFmpeg-devel] [PATCH]lavf/mov: Do not overread iTunes metadata

2015-04-03 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4425 for me. Please comment, Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index dc31e5a..8a4b739 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -342,7 +342,7 @@ retry: if (c->itunes_metadata && atom.size > 8) { int data_si

Re: [FFmpeg-devel] [PATCH]lavf/mov: Do not overread iTunes metadata

2015-04-03 Thread Carl Eugen Hoyos
On Saturday 04 April 2015 06:25:36 am Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4425 for me. New patch attached that does not brake fate. Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index dc31e5a..eda9d42 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.