Re: [FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

That seems logical, to set the fourcc for audio as well. I thought of it before, actually. Mats  -- Mats Peterson http://matsp888.no-ip.org/~mats/ From: Carl Eugen Hoyos To: FFmpeg development discussions and patches Sent: Friday, December

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

What is completely unacceptable? That I provide an improved patch that doesn't call mov.c? That was what another developer here complained about, so I fixed that. And I said before, I'm not going to "learn" how to use the messy Git system. Take it or leave it. Regarding "the original author",

[FFmpeg-devel] re?? compile ffmpeg errors

master branch lost some functions, such as /root/osg/osg/src/osgPlugins/ffmpeg/FFmpegDecoderAudio.cpp:230:20: error: ‘AVCodecContext’ has no member named ‘get_buffer’ m_context->get_buffer = avcodec_default_get_buffer; ^

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

On Friday 11 December 2015 12:32:13 pm Mats Peterson wrote: > OK, here is an improved version of my previous patch of > matroskadec.c that manages the palette itself without calling > code in mov.c. I have borrowed some code from it This is completely unacceptable afaict. Unrelated: It is a

[FFmpeg-devel] Fw: Fw: Fw: [PATCH] Make FFmpeg recognize QT version 0 sound sample descriptions and store the palette in matroskadec.c

I'm working on a better patch that manages the palette inside matroskadec.c. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ - Forwarded Message - From: Mats Peterson To: "ffmpeg-devel@ffmpeg.org" Sent: Friday,

Re: [FFmpeg-devel] compile ffmpeg errors

Wrong mailing list, but anyway: On Fri, Dec 11, 2015 at 15:09:23 +0800, 广州刘丹 wrote: > git checkout origin/release/2.8 Why not stick with master? > libavfilter/avf_showcqt.c:38:10: error: #include expects "FILENAME" or > Something went wrong with your libfreetype. (I think your ft2build.h is

[FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

OK, here is an improved version of my previous patch of matroskadec.c that manages the palette itself without calling code in mov.c. I have borrowed some code from it, though, and I'm including qtpalette.h. One thing I don't understand is why the minimum private data size for V_QUICKTIME has

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

Regarding the following statement by you: "Unrelated: It is a mystery to me how using the function that allows parsing stsd atoms to parse an stsd atom can be called a "hack", even more so when this is done in existing code." It's really a bad kludge, if you think about it. It's of course much

Re: [FFmpeg-devel] [PATCH] ffmpeg: change command line option -dump to work without -loglevel debug

On Thu, Dec 10, 2015 at 06:54:09PM +, Tom Marecek wrote: > -hex and -dump command line options do nothing unless -loglevel debug is set. > > -dump by itself is useful for monitoring live streams (to get the current PTS > for example) however when it is used with -loglevel debug for an RTMP

Re: [FFmpeg-devel] [PATCH 2/2] lavf: use a video frame pool for each link of the filtergraph

> Subject: [FFmpeg-devel] [PATCH 2/2] lavf: use a video frame pool for each > link of the filtergraph Since Ganesh just recently mentioned the confusion: Wouldn't this be lavfi instead of lavf? Yet another task for ./tools/patcheck. ;-) > libavfilter/avfilter.c | 1 + >

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

On Fri, Dec 11, 2015 at 11:48:15 +, Mats Peterson wrote: > And I said before, I'm not going to "learn" how to use the messy Git > system. What's messy? That's just the way it works here. It may be a bit more tedious for you, but makes life for everyone else (testers, reviewers, committers)

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

On Fri, 11 Dec 2015 12:59:16 + (UTC) Mats Peterson wrote: > I just want someone of you guys to take a look at what I've done. You > know Git much better than me, and you can create a pull request it > looks decent enough. I'm not going to participate a lot

Re: [FFmpeg-devel] [PATCH 0/3] Workaround for large JPEG over RTP

On Wed, Dec 9, 2015 at 4:26 PM, Carl Eugen Hoyos wrote: > On Tuesday 08 December 2015 01:10:30 pm Andrey Utkin wrote: >> RFCs for JPEG over RTP don't support dimensions larger than pixels, >> but it is possible to work around if we stick with information from SDP. > > Sorry if I

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

I just want someone of you guys to take a look at what I've done. You know Git much better than me, and you can create a pull request it looks decent enough. I'm not going to participate a lot in this in the future, but I wanted to correct these things, since they have been overlooked for such

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

Everyone except Carl, that is ;) Just kidding. Mats  -- Mats Peterson http://matsp888.no-ip.org/~mats/ From: Mats Peterson To: FFmpeg development discussions and patches Sent: Friday, December 11, 2015 1:59 PM Subject: Re:

Re: [FFmpeg-devel] Bug #3823 (yuvj422p jpeg rtp enc) request for help

On Wed, Dec 9, 2015 at 2:07 AM, Andrey Utkin wrote: > Hi! Could please anybody look at MJPEG RTP encoder issue with yuvj422p > https://trac.ffmpeg.org/ticket/3823#comment:17 ? This pixel format > becomes popular for IP cameras having no RTSP, and there are >

Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

Well, it's more or less an impossibility that anything will break from using 36 instead 86, since as I said before, 36 is the size of a version 0 sound sample description, which is rather old, but should of course still be recognized in FFmpeg. Let me refer to the QuickTime File Format

Re: [FFmpeg-devel] [PATCH 2/2] lavf: use a video frame pool for each link of the filtergraph

On Fri, Dec 11, 2015 at 01:32:47PM +0100, Matthieu Bouron wrote: [...] > diff --git a/libavfilter/video.c b/libavfilter/video.c > index 0274fc1..5b0b7f9 100644 > --- a/libavfilter/video.c > +++ b/libavfilter/video.c > @@ -32,6 +32,8 @@ > #include "internal.h" > #include "video.h" > > +#define

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

On 11.12.2015 17:21, Ganesh Ajjanagadde wrote: > On Fri, Dec 11, 2015 at 11:16 AM, Andreas Cadhalpun > wrote: >> On 19.11.2015 14:17, Michael Niedermayer wrote: >>> From: Michael Niedermayer >>> >>> Signed-off-by: Michael Niedermayer

[FFmpeg-devel] [PATCH 2/2] ffmpeg: Don't clear input pict_type if force_key_frames is source

The current behavior doesn't forward non-IDR I-frames to the encoder. Signed-off-by: Will Kelleher --- ffmpeg.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index a866f72..61739a4 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/aacsbr_template: Add Check to read_sbr_envelope()

On 19.11.2015 14:17, Michael Niedermayer wrote: > From: Michael Niedermayer > > The limit is a conservative guess, the spec does not seem to specify a limit > > Signed-off-by: Michael Niedermayer > --- > libavcodec/aacsbr_template.c | 41

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

On Fri, Dec 11, 2015 at 11:36 AM, Andreas Cadhalpun wrote: > On 11.12.2015 17:21, Ganesh Ajjanagadde wrote: >> On Fri, Dec 11, 2015 at 11:16 AM, Andreas Cadhalpun >> wrote: >>> On 19.11.2015 14:17, Michael Niedermayer wrote:

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

On Thu, Dec 10, 2015 at 10:29 AM, Michael Niedermayer wrote: > On Wed, Dec 09, 2015 at 07:02:02PM -0500, Ganesh Ajjanagadde wrote: >> exp2 suffices here. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/nellymoserdec.c | 2 +- >> 1 file

Re: [FFmpeg-devel] [PATCH 7/9] avcodec/mdct_template: use lrint instead of floor hack

On Thu, Dec 10, 2015 at 8:54 AM, Ganesh Ajjanagadde wrote: > On Sun, Dec 6, 2015 at 8:20 AM, Ganesh Ajjanagadde > wrote: >> On Tue, Dec 1, 2015 at 7:27 PM, Ganesh Ajjanagadde >> wrote: >>> Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] ffmpeg: add progress speed to status line and report

This adds a computation of the progress speed versus realtime ("Nx") to the status line and to the report log. It uses the progress time as already calculated for total output time as a base. --- ffmpeg.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/ffmpeg.c

Re: [FFmpeg-devel] [PATCH] avutil/softfloat: Assert that the exponent did not overflow the legal range in av_normalize1_sf()

On 11.12.2015 00:48, Michael Niedermayer wrote: > On Mon, Nov 09, 2015 at 09:17:34PM +0100, Andreas Cadhalpun wrote: >> On 08.11.2015 21:51, Andreas Cadhalpun wrote: >>> On 08.11.2015 13:41, Michael Niedermayer wrote: From: Michael Niedermayer Signed-off-by:

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

On 19.11.2015 14:17, Michael Niedermayer wrote: > From: Michael Niedermayer > > Signed-off-by: Michael Niedermayer > --- > libavcodec/aacsbr.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/aacsbr: Split the env_facs table

On 19.11.2015 14:17, Michael Niedermayer wrote: > From: Michael Niedermayer > > This also removes a #ifdef and special case for the fixed point case > > Signed-off-by: Michael Niedermayer > --- > libavcodec/aacsbr.c |6 +++--- >

[FFmpeg-devel] [PATCH] avfilter: add SOFAlizer audio filter

Signed-off-by: Paul B Mahol --- configure | 4 + doc/filters.texi | 29 ++ libavfilter/Makefile | 1 + libavfilter/af_sofalizer.c | 967 + libavfilter/allfilters.c | 1 + libavfilter/formats.c

Re: [FFmpeg-devel] [PATCHv2] avcodec/dcaenc: avoid wasteful cos calls

On Fri, Dec 11, 2015 at 3:29 AM, Paul B Mahol wrote: > On 12/10/15, Ganesh Ajjanagadde wrote: >> cos has symmetry; use this. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- >> libavcodec/dcaenc.c | 10 +- >> 1 file

[FFmpeg-devel] [PATCH] ffmpeg: add progress speed to status line and report

ffmpeg.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) While this is still not a benchmark, the display of a factor "Nx" gives a much better and easier to parse feeling for the processing speed than looking at "fps", at the processed file time, or applying the Unix "time"

Re: [FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

On Fri, Dec 11, 2015 at 09:32:23AM +0100, Carl Eugen Hoyos wrote: > On Friday 11 December 2015 02:29:53 am Michael Niedermayer wrote: > > On Thu, Dec 10, 2015 at 11:50:45PM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch is definitely a good idea imo, the mov demuxer also > > >

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

On 11.12.2015 09:41, Carl Eugen Hoyos wrote: > On Friday 11 December 2015 12:16:48 am Andreas Cadhalpun wrote: >> Well, the problem is that the answer may depend on the system. > > If Nvidia offers Graphics Driver for download on its ftp server > for multiple operating systems, they are either

Re: [FFmpeg-devel] [PATCH 1/3] avformat: Add av_program_add_stream_index()

On Thu, Dec 10, 2015 at 09:54:02PM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > This will be used by the subsequent commit(s) > > Signed-off-by: Michael Niedermayer > --- > doc/APIchanges |3 +++ >

Re: [FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

This patch seems to be causing some issues with some h264 video-only content. Attempting to play the following in ffplay with "-v debug -drp 1" (disables pts/dts fixups and purely uses pts) will cause jerky playback after seeking around a few times due to dropped out of order frames. These are

Re: [FFmpeg-devel] [PATCH] avcodec/golomb: Mask shift amount before use in get_ue_golomb()

On Fri, Dec 11, 2015 at 4:14 PM, Andreas Cadhalpun wrote: > On 07.12.2015 00:27, Ganesh Ajjanagadde wrote: >> On Sun, Dec 6, 2015 at 6:12 PM, Andreas Cadhalpun >> wrote: >>> On 06.12.2015 22:48, Michael Niedermayer wrote:

Re: [FFmpeg-devel] [PATCH] avcodec/golomb: Mask shift amount before use in get_ue_golomb()

On 07.12.2015 00:27, Ganesh Ajjanagadde wrote: > On Sun, Dec 6, 2015 at 6:12 PM, Andreas Cadhalpun > wrote: >> On 06.12.2015 22:48, Michael Niedermayer wrote: >>> my concern is more on h264 (CAVLC) and hevc speed >> >> I tested with 444_8bit_cavlc.h264 added 100

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

On 2015-12-12 00:03, Andreas Cadhalpun wrote: On 11.12.2015 09:41, Carl Eugen Hoyos wrote: On Friday 11 December 2015 12:16:48 am Andreas Cadhalpun wrote: Well, the problem is that the answer may depend on the system. If Nvidia offers Graphics Driver for download on its ftp server for

[FFmpeg-devel] MPlayer woes (and an explanation of my patch of matroskadec.c)

I have noticed that MPlayer blindly regards anything in the 'extradata' for a video stream as being a palette, that it tacks onto a "fake" BITMAPINFOHEADER in libmpdemux/demux_lavf.c, and which is used later on as palette side data in libmpcodecs/vd_ffmpeg.c. In my patch of

Re: [FFmpeg-devel] [PATCH] avfilter/formats: fix segfault when allocation fails

On Tue, Dec 8, 2015 at 9:57 PM, Ganesh Ajjanagadde wrote: > On Sat, Dec 5, 2015 at 2:43 PM, Ganesh Ajjanagadde > wrote: >> This is a somewhat subtle failure that can occur when the realloc_array >> fails in FORMATS_REF. >> >> Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCHv2] avcodec/dcaenc: avoid wasteful cos calls

On 12/10/15, Ganesh Ajjanagadde wrote: > cos has symmetry; use this. > > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/dcaenc.c | 10 +- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/dcaenc.c

Re: [FFmpeg-devel] [PATCH]lavf/matroskadec: Set codec_tag also for audio codecs

On Friday 11 December 2015 02:29:53 am Michael Niedermayer wrote: > On Thu, Dec 10, 2015 at 11:50:45PM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch is definitely a good idea imo, the mov demuxer also > > sets codec_tag reading the same atom, for "A_MS/ACM" codec_tag is > > already

[FFmpeg-devel] Correct use of zoompan filter in combination with other filters.

Hoping a developer can enlighten me as to why I'm having problems chaining the zoompan filter to the overlay filter. When I take a single jpg, feed it into zoompan and conjure 250 frames of output it works fine. But when I attempt to take those 250 frames and use them as an input to the overlay

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

On Friday 11 December 2015 12:16:48 am Andreas Cadhalpun wrote: > On 11.12.2015 00:03, Carl Eugen Hoyos wrote: > > On Monday 07 December 2015 07:53:31 pm Timo Rothenpieler wrote: > >> @@ -4807,7 +4807,6 @@ die_license_disabled gpl x11grab > >> > >> die_license_disabled nonfree libaacplus > >>

Re: [FFmpeg-devel] Correct use of zoompan filter in combination with other filters.

On 12/11/15, Ryan Williams wrote: > Hoping a developer can enlighten me as to why I'm having problems chaining > the zoompan filter to the overlay filter. > > When I take a single jpg, feed it into zoompan and conjure 250 frames of > output it works fine. > But when I

Re: [FFmpeg-devel] [PATCH] nvenc set slice number to 1 to improve encoding quality and clamp initial qp value to [1, 51]

> Hi all, before switching to the new released nvenc6.0 header, can you > take a look at this fix? I'm fine with making it the default, the only concern i have is weather it has any side-effects, like reduced performance that impacts some higher resolutions which can't be encoded in realtime

Re: [FFmpeg-devel] [PATCH]lavc/libvo-aac: Mark the encoder as experimental

On Wed, 9 Dec 2015 21:24:06 +0100, Hendrik Leppkes wrote: > libvo-aacenc doesn't see any development anymore. > Its an encoder taken from the android sdk a long time ago, but AOSP > replaced it with fdk-aac, so libvo-aacenc is EOL. > > And its quality is horrible. We should remove the wrapper.

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: Don't clear input pict_type if force_key_frames is source

On Fri, Dec 11, 2015 at 09:47:29AM -0600, Will Kelleher wrote: > The current behavior doesn't forward non-IDR I-frames to the encoder. would this not turn every I-type non keyframe into a key frame effectively? and if so, is that intended ? [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/aacsbr: Add comment about possibly optimization in sbr_dequant()

On 11.12.2015 18:09, Ganesh Ajjanagadde wrote: > On Fri, Dec 11, 2015 at 11:36 AM, Andreas Cadhalpun > wrote: >> On 11.12.2015 17:21, Ganesh Ajjanagadde wrote: >>> On Fri, Dec 11, 2015 at 11:16 AM, Andreas Cadhalpun >>> wrote:

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add AVVideoFramePool

On Fri, Dec 11, 2015 at 01:32:46PM +0100, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavutil/frame.c | 153 > + > libavutil/frame.h | 56 + > libavutil/frame_internal.h

Re: [FFmpeg-devel] [PATCH] avfilter: add SOFAlizer audio filter

On 12/11/15, Michael Niedermayer wrote: > On Fri, Dec 11, 2015 at 12:03:34AM +0100, Paul B Mahol wrote: > [...] >> +static int query_formats(AVFilterContext *ctx) >> +{ >> +struct SOFAlizerContext *s = ctx->priv; >> +AVFilterFormats *formats = NULL; >> +

[FFmpeg-devel] [PATCH 1/2] lavu: add AVVideoFramePool

From: Matthieu Bouron --- libavutil/frame.c | 153 + libavutil/frame.h | 56 + libavutil/frame_internal.h | 37 +++ 3 files changed, 246 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH 2/2] lavf: use a video frame pool for each link of the filtergraph

From: Matthieu Bouron --- libavfilter/avfilter.c | 1 + libavfilter/avfilter.h | 5 + libavfilter/video.c| 38 +++--- 3 files changed, 33 insertions(+), 11 deletions(-) diff --git a/libavfilter/avfilter.c

[FFmpeg-devel] Add a frame/buffer pool to AVFilter

--- Hello, The following two commits aim to add a video frame pool to AVFilter. This feature was originally introduced by commit 39f66edbeae5ccabefe38b2fcb25d6c242d868c0 but was removed later on when the AVFilter API was changed to use AVFrame. The AVVideoFramePool implementation is similar to

Re: [FFmpeg-devel] Add a frame/buffer pool to AVFilter

On Fri, Dec 11, 2015 at 01:32:45PM +0100, Matthieu Bouron wrote: > --- > > Hello, > > The following two commits aim to add a video frame pool to AVFilter. > This feature was originally introduced by commit > 39f66edbeae5ccabefe38b2fcb25d6c242d868c0 but was removed later on when the > AVFilter

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: Don't clear input pict_type if force_key_frames is source

On 12/11, Michael Niedermayer wrote: > On Fri, Dec 11, 2015 at 09:47:29AM -0600, Will Kelleher wrote: > > The current behavior doesn't forward non-IDR I-frames to the encoder. > > would this not turn every I-type non keyframe into a key frame > effectively? > and if so, is that intended ? >

Re: [FFmpeg-devel] [PATCH 1/2] lavu: add AVVideoFramePool

On Fri, Dec 11, 2015 at 7:07 PM, Michael Niedermayer wrote: > On Fri, Dec 11, 2015 at 01:32:46PM +0100, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > --- > > libavutil/frame.c | 153 >

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: Don't clear input pict_type if force_key_frames is source

On Fri, Dec 11, 2015 at 02:23:56PM -0600, Will Kelleher wrote: > On 12/11, Michael Niedermayer wrote: > > On Fri, Dec 11, 2015 at 09:47:29AM -0600, Will Kelleher wrote: > > > The current behavior doesn't forward non-IDR I-frames to the encoder. > > > > would this not turn every I-type non