Re: [FFmpeg-devel] [PATCH] lavf/mpegtsenc: add special case for handling timed ID3 packets

2016-07-29 Thread Stefano Sabatini
On date Thursday 2016-05-19 22:20:41 +0200, Michael Niedermayer encoded: > On Thu, May 19, 2016 at 06:45:41PM +0200, Stefano Sabatini wrote: > > Set the stream_id to 0xbd (private_stream_id_1). Tools seem to assume > > that value, and this is consistent with MPEG TS (ITU-T H.222.0) section > > 2.12

Re: [FFmpeg-devel] [PATCH] hapdec: remove unused memory.h include

2016-07-29 Thread Josh de Kock
On Fri, Jul 29, 2016, at 03:49 AM, Timothy Gu wrote: > Looks good to me. For what it's worth, the include was added in > commit 3ee217853a6741b829a2683f49c590618891b1ab, and looks like a stray > change. > > Timothy Thanks, applied. Josh ___ ffmpeg-deve

Re: [FFmpeg-devel] [PATCH] Fix double free and null dereferences in the qsv decoder

2016-07-29 Thread Ivan Uskov
Hello Yuli, Wednesday, July 27, 2016, 6:21:41 PM, you wrote: > This patch fixes the h264_qsv decoder issues mentioned > in https://ffmpeg.zeranoe.com/forum/viewtopic.php?t=2962. > The patch may be tested by specifying h264_qsv as the decoder to ffplay > for an h264 encoded file. > ffpl

Re: [FFmpeg-devel] [PATCH] Fix audio clipping problem in dynaudnorm

2016-07-29 Thread Paul B Mahol
On 7/27/16, andyndeanna wrote: > Hello, > > See attached patch to fix a clipping problem with dynaudnorm when the first > frame contains silence. I implemented the fix by adding a boundary mode. > It could also be done by changing how b=1 works. > > Thanks, > Andy > Have you considered sending t

[FFmpeg-devel] [PATCH] avcodec/alacenc: allocate bigger packets

2016-07-29 Thread Paul B Mahol
Hi, patch attached. 0001-avcodec-alacenc-allocate-bigger-packets.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: implement floating point decoding

2016-07-29 Thread Umair Khan
On Thu, Jul 28, 2016 at 7:12 PM, Umair Khan wrote: > On Thu, Jul 28, 2016 at 2:25 PM, Thilo Borgmann > wrote: >> Am 28.07.16 um 08:28 schrieb Umair Khan: >>> On Thu, Jul 28, 2016 at 12:22 AM, Clément Bœsch wrote: On Wed, Jul 27, 2016 at 07:48:56PM +0200, Thilo Borgmann wrote: >> @@ -18

Re: [FFmpeg-devel] [PATCH] Fix double free and null dereferences in the qsv decoder

2016-07-29 Thread Yuli Khodorkovskiy
On Fri, Jul 29, 2016 at 5:35 AM, Ivan Uskov wrote: > > Hello Yuli, > > Wednesday, July 27, 2016, 6:21:41 PM, you wrote: > > > This patch fixes the h264_qsv decoder issues mentioned > > in https://ffmpeg.zeranoe.com/forum/viewtopic.php?t=2962. > > > The patch may be tested by specifying h264_qsv a

[FFmpeg-devel] [PATCH] Fix null dereferences in the qsv decoder

2016-07-29 Thread Yuli Khodorkovskiy
This patch fixes the h264_qsv decoder issues mentioned in https://ffmpeg.zeranoe.com/forum/viewtopic.php?t=2962. The patch may be tested by specifying h264_qsv as the decoder to ffplay for an h264 encoded file. ffplay -vcodec h264_qsv foo.mts Signed-off-by: Yuli Khodorkovskiy --- libav

[FFmpeg-devel] [PATCH 0/7] fix(v2) for #5727 split up, plus other things

2016-07-29 Thread Burt P
Here is a set with the earlier patch split into smaller patches. Added to the old version is hdcd_control_stereo() that logs information about the mismatched target_gain so that problem samples can be found more easily. Also, a patch for warning about problems in the AVFilterLink chain. Thanks f

[FFmpeg-devel] [PATCH 5/7] af_hdcd: Move code detect/check into own function

2016-07-29 Thread Burt P
Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 86 +++ 1 file changed, 60 insertions(+), 26 deletions(-) diff --git a/libavfilter/af_hdcd.c b/libavfilter/af_hdcd.c index aeb65eb..902827e 100644 --- a/libavfilter/af_hdcd.c +++ b/libavfilter/af_hd

[FFmpeg-devel] [PATCH 6/7] af_hdcd: Process stereo channels together, fix #5727

2016-07-29 Thread Burt P
Issue #5727: gain adjustment should only be applied if matching gain value from a valid packet in both channels. The existing functions process each channel separately, so it was not possible. * New versions of hdcd_process(), hdcd_scan(), hdcd_integrate() named hdcd_*_stereo() that process both

[FFmpeg-devel] [PATCH 1/7] af_hdcd: fix a minor annoyance

2016-07-29 Thread Burt P
Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/af_hdcd.c b/libavfilter/af_hdcd.c index 6f3eb1e..b860077 100644 --- a/libavfilter/af_hdcd.c +++ b/libavfilter/af_hdcd.c @@ -23,12 +23,12 @@ LIABILITY, WHETHER IN

[FFmpeg-devel] [PATCH 2/7] af_hdcd: give cdt expired counter a value for never set

2016-07-29 Thread Burt P
The counter is now -1 if the code detect timer was never set, and 0 if it was set but never expired. Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/libavfilter/af_hdcd.c b/libavfilter/af_hdcd.c index b860077.

[FFmpeg-devel] [PATCH 3/7] af_hdcd: Improve error detection logging

2016-07-29 Thread Burt P
* Moves the filter context member out of state and into HDCDContext * More useful information when an error is detected * Gives a location near where the error was detected Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 34 +++--- 1 file changed, 19 insertions(+),

[FFmpeg-devel] [PATCH 7/7] af_hdcd: Warn if there is any resampling or format conversion in the link chain

2016-07-29 Thread Burt P
HDCD is only encoded in s16@44100Hz. Scan the chain of AVFilterLinks for any resampling or format conversion/truncation that might cause problems for the filter and issue warnings. Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 62 +++ 1 file ch

[FFmpeg-devel] [PATCH 4/7] af_hdcd: add force_pe filter option

2016-07-29 Thread Burt P
Used to attempt replication of some results from http://www.audiomisc.co.uk/HFN/HDCD/Examined.html May not be generally useful, defaults to off. Signed-off-by: Burt P --- libavfilter/af_hdcd.c | 26 ++ 1 file changed, 22 insertions(+), 4 deletions(-) diff --git a/libavfi

Re: [FFmpeg-devel] [PATCH] Fix audio clipping problem in dynaudnorm

2016-07-29 Thread andyndeanna
Paul, I didn't, but that makes sense. Thanks, Andy On Friday, July 29, 2016, Paul B Mahol wrote: > On 7/27/16, andyndeanna > wrote: > > Hello, > > > > See attached patch to fix a clipping problem with dynaudnorm when the > first > > frame contains silence. I implemented the fix by adding a bo

Re: [FFmpeg-devel] [PATCH] avfilter: Add new format for PSNR stats log

2016-07-29 Thread Michael Niedermayer
On Mon, Jul 25, 2016 at 11:54:37AM -0700, Lucas Cooper wrote: > Add an AVOption stats_version with a new header for V2 stats, which > specifies the stats log version and lists the fields that will be > present in the log (to ease parsing). > > The primary motivation is to facilitate the addition o

Re: [FFmpeg-devel] [PATCH] libavformat/matroskadec: Add test for seeking with codec delay.

2016-07-29 Thread Michael Niedermayer
On Fri, Jul 29, 2016 at 12:54:13AM +0200, Michael Niedermayer wrote: > On Wed, Jul 27, 2016 at 06:33:30PM -0700, chcunning...@chromium.org wrote: > > From: Chris Cunningham > > > > Also cleanup parens for the skip_to_timecode check. > > --- > > libavformat/matroskadec.c | 2 +- > > tests/f

[FFmpeg-devel] [PATCH] dynarray: Change AV_ to FF_ for AV_DYNARRAY_ADD

2016-07-29 Thread Timothy Gu
The header is not installed and the macro isn't used outside libavutil, so it is obviously privat to libavutil. Make the name reflect that. --- libavutil/dynarray.h | 2 +- libavutil/mem.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavutil/dynarray.h b/libavu

[FFmpeg-devel] [PATCH 2/2] mem: Make function attribute usage consisten

2016-07-29 Thread Timothy Gu
--- libavutil/mem.h | 42 +++--- 1 file changed, 27 insertions(+), 15 deletions(-) diff --git a/libavutil/mem.h b/libavutil/mem.h index 2f53b47..ca936e8 100644 --- a/libavutil/mem.h +++ b/libavutil/mem.h @@ -80,7 +80,8 @@ * be allocated. * @see av_mallocz()

[FFmpeg-devel] [PATCH 1/2] build: Make API documentation depend on config.mak

2016-07-29 Thread Timothy Gu
The Doxygen command is generated from the list of installed headers, which may change per configuration (e.g. `--enable-gpl` results in libpostproc to be built and installed). --- doc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/Makefile b/doc/Makefile index 4a7