Re: [FFmpeg-devel] Odp: Re: [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Lou Logan
On Wed, Aug 30, 2017, at 11:53 AM, samsamsam wrote: > [...] Please fix your emails—they are broken. Here's what one looks like: Your messages are: * At least 150k in size. There is no need for them to be this large.

Re: [FFmpeg-devel] [PATCH v3] fate: add tests for some video source filters

2017-08-30 Thread Michael Niedermayer
On Tue, Aug 29, 2017 at 04:49:58PM +0200, Tobias Rapp wrote: > On 25.07.2017 11:11, Tobias Rapp wrote: > >Adds FATE tests for the previously untested allrgb, allyuv, rgbtestsrc, > >smptebars, smptehdbars and yuvtestsrc filters. > > > >Also adds a test for testsrc2 filter with rgb+alpha. > > >

Re: [FFmpeg-devel] fate/pixlet : add test for rgb

2017-08-30 Thread Michael Niedermayer
On Mon, Aug 28, 2017 at 05:51:48PM +0200, Martin Vignali wrote: > 2017-04-03 2:21 GMT+02:00 Michael Niedermayer : > > > On Sun, Apr 02, 2017 at 11:29:38PM +0200, Martin Vignali wrote: > > > Hello, > > > > > > In attach patch to add a fate test for pixlet decoder > > > > >

Re: [FFmpeg-devel] [PATCH] avfilter/mcdeint: remove usage of deprecated AVCodecContext.me_method

2017-08-30 Thread James Almer
On 8/30/2017 9:39 PM, Michael Niedermayer wrote: > On Mon, Aug 28, 2017 at 02:18:53PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> Depends on patch "avcodec/snowenc: fix setting motion_est option" >> >> libavfilter/vf_mcdeint.c | 3 ++- >> 1 file changed,

Re: [FFmpeg-devel] [PATCH] avfilter/mcdeint: remove usage of deprecated AVCodecContext.me_method

2017-08-30 Thread Michael Niedermayer
On Mon, Aug 28, 2017 at 02:18:53PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > Depends on patch "avcodec/snowenc: fix setting motion_est option" > > libavfilter/vf_mcdeint.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] avformat/utils: refine av_get_frame_filename2 function

2017-08-30 Thread Michael Niedermayer
On Mon, Aug 28, 2017 at 08:03:36PM +0800, Steven Liu wrote: > support parse the l for the %lld or %ld > > Signed-off-by: Steven Liu > --- > libavformat/utils.c | 4 > 1 file changed, 4 insertions(+) LGTM please add a fate test for this. thx [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH]lavc/hevc_ps: Be less strict on sps conformance by default.

2017-08-30 Thread Michael Niedermayer
On Wed, Aug 30, 2017 at 12:05:32PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows users to control the sps conformance > requirement: It allows to revert the fix for ticket #4035, so faulty > streams can be ignored. > Also fixes a user-provided sample. > > Please comment, Carl

Re: [FFmpeg-devel] [PATCH 1/2] avformat/flvdec: Set need_context_update when setting the initial extradata

2017-08-30 Thread Michael Niedermayer
On Tue, Aug 29, 2017 at 11:40:06AM -0700, Alex Converse wrote: > Fixes ticket 6398. > > Debugged with the help of James Almer and Hendrik Leppkes. > --- > libavformat/flvdec.c | 1 + > 1 file changed, 1 insertion(+) should be ok thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] libavformat/dv : read dv audio as BE

2017-08-30 Thread Michael Niedermayer
On Wed, Aug 30, 2017 at 02:27:36PM +0300, Александр Слободенюк wrote: > #include "libavutil/bswap.h" was not necessary for the compilation. > > > I'm really sorry for the waste of your time... very shameful, that I > > didn't mind to run fate first.. > > > The patch had 2 different bugs at the

Re: [FFmpeg-devel] [PATCHv3 2/2] avcodec/vp9: Add tile threading support

2017-08-30 Thread gh0st
Should be fixed now in PATCHv4, thanks! On Thu, Aug 31, 2017 at 3:59 AM, Michael Niedermayer wrote: > On Mon, Aug 28, 2017 at 06:36:13PM +0700, Ilia Valiakhmetov wrote: > > Signed-off-by: Ilia Valiakhmetov > > --- > > libavcodec/vp9.c |

Re: [FFmpeg-devel] [PATCH] lavfi/atempo: Avoid false triggering an assertion failure

2017-08-30 Thread Pavel Koshevoy
On Wed, Aug 30, 2017 at 2:00 PM, Marton Balint wrote: > > On Mon, 19 Dec 2016, Marton Balint wrote: > >> >> On Sat, 17 Dec 2016, pkoshe...@gmail.com wrote: >> >>> From: Pavel Koshevoy >>> >>> Steps to reproduce: >>> ./ffmpeg_g -f s16be -i /dev/null -af

Re: [FFmpeg-devel] [PATCHv3 2/2] avcodec/vp9: Add tile threading support

2017-08-30 Thread Michael Niedermayer
On Mon, Aug 28, 2017 at 06:36:13PM +0700, Ilia Valiakhmetov wrote: > Signed-off-by: Ilia Valiakhmetov > --- > libavcodec/vp9.c | 608 > ++- > libavcodec/vp9_mc_template.c | 202 +++--- > libavcodec/vp9block.c

Re: [FFmpeg-devel] [PATCH] lavfi/atempo: Avoid false triggering an assertion failure

2017-08-30 Thread Marton Balint
On Mon, 19 Dec 2016, Marton Balint wrote: On Sat, 17 Dec 2016, pkoshe...@gmail.com wrote: From: Pavel Koshevoy Steps to reproduce: ./ffmpeg_g -f s16be -i /dev/null -af atempo=0.5 -y /tmp/atempo.wav --- libavfilter/af_atempo.c | 4 ++-- 1 file changed, 2 insertions(+),

Re: [FFmpeg-devel] [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Rodger Combs
I'm not confident that we'll be able to verify the correctness of any validation code for a printf format string sufficiently. Just parse out the padding value and use "%0*"PRId64. Do not pass anything other than a string literal to printf-family functions in future versions of this patch. >

[FFmpeg-devel] [PATCH] nvenc: Don't segfault on close if no cuda is available

2017-08-30 Thread Mark Thompson
--- """ Cannot load libcuda.so.1 Thread 1 "ffmpeg_g" received signal SIGSEGV, Segmentation fault. 0x5657cd59 in ff_nvenc_encode_close (avctx=0x578ed920) at src/libavcodec/nvenc.c:1337 1337cu_res = dl_fn->cuda_dl->cuCtxPushCurrent(ctx->cu_context); """ This only fixes the

Re: [FFmpeg-devel] [PATCH] avfilter: add ADM filter

2017-08-30 Thread Ashish Pratap Singh
Hi, On Aug 30, 2017 22:52, "Paul B Mahol" wrote: On 8/30/17, Ashish Pratap Singh wrote: > From: Ashish Singh > > Hi, > This patch changes the previous adm filter to integer types which will be > helpful for writing SIMD optimizations

[FFmpeg-devel] [PATCH] avfilter/vf_subtitles: enable processing of alpha channel

2017-08-30 Thread Paul B Mahol
Fixes #6605. Signed-off-by: Paul B Mahol --- libavfilter/vf_subtitles.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 805ee2d..b7ba628 100644 --- a/libavfilter/vf_subtitles.c +++

Re: [FFmpeg-devel] [PATCH] avfilter: add ADM filter

2017-08-30 Thread Paul B Mahol
On 8/30/17, Ashish Pratap Singh wrote: > From: Ashish Singh > > Hi, > This patch changes the previous adm filter to integer types which will be > helpful for writing SIMD optimizations later. > > Signed-off-by: Ashish Singh > --- >

[FFmpeg-devel] [PATCH] avfilter: add ADM filter

2017-08-30 Thread Ashish Pratap Singh
From: Ashish Singh Hi, This patch changes the previous adm filter to integer types which will be helpful for writing SIMD optimizations later. Signed-off-by: Ashish Singh --- Changelog| 1 + doc/filters.texi | 15 +

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: add fallback for calculating file duration

2017-08-30 Thread Nicolas George
Le primidi 11 fructidor, an CCXXV, Justin Ruggles a écrit : > If a file does not have a known duration, this leads to the timestamps > starting over for the next file, causing non-monotonic timestamps. > To prevent this, track the duration during demuxing and use it to > determine the current file

Re: [FFmpeg-devel] [PATCH] avfilter/vf_libvmaf: fix pre convert to framesync2 bugs

2017-08-30 Thread Ashish Pratap Singh
Hi, On 30-08-2017 19:43, Derek Buitenhuis wrote: On 8/30/2017 2:24 PM, Ronald S. Bultje wrote: Pushed. Did libvmaf bump its version for this breaking change? (It should have...) If so, bump the required version in configure, or other users will get crashes. - Derek

Re: [FFmpeg-devel] [PATCH] avformat/concatdec: add fallback for calculating file duration

2017-08-30 Thread Paul B Mahol
On 8/29/17, Justin Ruggles wrote: > If a file does not have a known duration, this leads to the timestamps > starting over for the next file, causing non-monotonic timestamps. > To prevent this, track the duration during demuxing and use it to > determine the current

Re: [FFmpeg-devel] [PATCH] avfilter/vf_libvmaf: fix pre convert to framesync2 bugs

2017-08-30 Thread Derek Buitenhuis
On 8/30/2017 2:24 PM, Ronald S. Bultje wrote: > Pushed. Did libvmaf bump its version for this breaking change? (It should have...) If so, bump the required version in configure, or other users will get crashes. - Derek ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] h264: Make Truncating SPS/PPS message debug.

2017-08-30 Thread Kieran Kunhya
Padded SPS/PPS is perfectly legal. 0001-h264-Make-Truncating-SPS-PPS-message-debug.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 10/16] lavfi/vf_libvmaf: convert to framesync2.

2017-08-30 Thread Ronald S. Bultje
Hi, On Sun, Aug 20, 2017 at 5:52 AM, Nicolas George wrote: > Le tridi 23 thermidor, an CCXXV, Nicolas George a écrit : > > After this commit, the code compiles, but on my setup it > > segfaults before and after. It also prints the very worrying > > warning: > > > >

Re: [FFmpeg-devel] [PATCH] avfilter/vf_libvmaf: fix pre convert to framesync2 bugs

2017-08-30 Thread Ronald S. Bultje
Hi, On Thu, Aug 3, 2017 at 1:34 AM, Ashish Pratap Singh wrote: > From: Ashish Singh > > Hi, it fixes the errors while converting to framesync2. > libvmaf was changed recently, double *score variable is removed in the new > version since it's not used

Re: [FFmpeg-devel] [PATCH] configure: require pkg-config for libvmaf

2017-08-30 Thread Ronald S. Bultje
Hi, On Tue, Aug 1, 2017 at 11:14 AM, Ashish Pratap Singh wrote: > From: Ashish Singh > > This patch makes the libvmaf filter use pkg-config to detect > and link to libvmaf. > > Signed-off-by: Ashish Singh > --- > configure | 3

[FFmpeg-devel] [PATCH v20] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Steven Liu
ffmpeg need a dash demuxer for demux the dash formats base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch TODO: 1. make the safety APIs used same to dashenc. 2. support multi bitrate dash. v2 fixed: 1. from autodetect to

Re: [FFmpeg-devel] Switching of automatically detected libraries [v2]

2017-08-30 Thread Clément Bœsch
On Wed, Aug 30, 2017 at 02:08:59PM +0200, Clément Bœsch wrote: > This patchset includes the following changes from previously: > > - The combination of --disable-autodetect and --enable- with > an automatically detected library now works > - Linking against libc's iconv is now supported with >

Re: [FFmpeg-devel] [PATCH v17] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Steven Liu
2017-08-30 19:50 GMT+08:00 Timo Rothenpieler : >>> Just wanted to point out, as stated on IRC, that dashenc already has a >>> function that seems to safely solve the filename template issue, and >>> looks >>> like it can be moved into a common file + header, and reused as

[FFmpeg-devel] [PATCH 07/16] build: treat sdl2 like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 94b95f2df7..911b9ae569 100755 --- a/configure +++ b/configure @@ -6026,20 +6026,22 @@ if enabled gcrypt; then fi fi -if !

[FFmpeg-devel] [PATCH 06/16] build: isolate sdl-to-sdl2 aliasing

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch This simplifies incoming SDL related changes by removing potential mismatching states of sdl and sdl2 variables. Since a component can have all kind of states (such as unset, enabled, disabled or requested), keeping these variables in sync manually in

[FFmpeg-devel] [PATCH 13/16] build: make alsa part of the autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch alsa libs are already autodetected, this commit makes sure --disable-autodetect actually disable it unless --enable-alsa is specified. --- configure| 6 +++--- libavdevice/Makefile | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff

[FFmpeg-devel] [PATCH 15/16] build: make sndio part of the autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch sndio is already autodetected, this commit makes sure --disable-autodetect actually disable it unless --enable-sndio is specified. --- configure| 5 +++-- libavdevice/Makefile | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git

[FFmpeg-devel] [PATCH 16/16] build: drop unused sndio_h and asoundlib_h

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 2 -- 1 file changed, 2 deletions(-) diff --git a/configure b/configure index f2af0711e3..6a50f40517 100755 --- a/configure +++ b/configure @@ -1854,7 +1854,6 @@ HAVE_LIST_PUB=" " HEADERS_LIST=" -alsa_asoundlib_h altivec_h

[FFmpeg-devel] [PATCH 12/16] build: make sure a disabled autodetect still pick the libc's iconv

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/configure b/configure index a6bcee9466..0a961842df 100755 --- a/configure +++ b/configure @@ -3592,6 +3592,11 @@ for e in $env; do done if disabled

[FFmpeg-devel] [PATCH 14/16] build: make jack part of the autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch jack is already autodetected, this commit makes sure --disable-autodetect actually disable it unless --enable-jack is specified. --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index

[FFmpeg-devel] [PATCH 11/16] build: add --disable-autodetect switch

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- Changelog | 1 + configure | 6 ++ 2 files changed, 7 insertions(+) diff --git a/Changelog b/Changelog index b064328c3f..8b6057d6ae 100644 --- a/Changelog +++ b/Changelog @@ -38,6 +38,7 @@ version : - Some video filters with several inputs now

[FFmpeg-devel] [PATCH 10/16] build: simplify weak-enabling of autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 18 +++--- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 7e80074686..e376e9562b 100755 --- a/configure +++ b/configure @@ -3593,21 +3593,9 @@ done for lib in $AUTODETECT_LIBS; do

[FFmpeg-devel] [PATCH 09/16] build: remove vda_framework from enable_weak

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch vda_framework is already pulled by the weakly enabled vda and videotoolbox. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 42742c5842..7e80074686 100755 --- a/configure +++ b/configure @@

[FFmpeg-devel] [PATCH 05/16] build: treat securetransport and schannel like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/configure b/configure index e73773fb74..5d416d97f4 100755 --- a/configure +++ b/configure @@ -3606,6 +3606,7 @@ enable_weak cuda cuvid nvenc vda_framework

[FFmpeg-devel] [PATCH 04/16] build: treat libxcb like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 24 +--- 1 file changed, 5 insertions(+), 19 deletions(-) diff --git a/configure b/configure index b4b1288722..e73773fb74 100755 --- a/configure +++ b/configure @@ -3598,7 +3598,7 @@ enable_weak audiotoolbox # Enable

[FFmpeg-devel] [PATCH 08/16] build: replace use of HAVE_SDL2 with existing CONFIG_SDL2

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch There is no need for duplication. --- configure| 1 - libavdevice/opengl_enc.c | 22 +++--- 2 files changed, 11 insertions(+), 12 deletions(-) diff --git a/configure b/configure index 911b9ae569..42742c5842 100755 ---

[FFmpeg-devel] [PATCH 03/16] build: treat iconv like other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index a9932aeba6..b4b1288722 100755 --- a/configure +++ b/configure @@ -3605,6 +3605,8 @@ enable_weak cuda cuvid nvenc vda_framework videotoolbox

[FFmpeg-devel] Switching of automatically detected libraries [v2]

2017-08-30 Thread Clément Bœsch
This patchset includes the following changes from previously: - The combination of --disable-autodetect and --enable- with an automatically detected library now works - Linking against libc's iconv is now supported with --disable-autodetect (issue raised by Nicolas) - SDL detection should be

[FFmpeg-devel] [PATCH 02/16] build: treat crystalhd like other hwaccels

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 44e45791c2..a9932aeba6 100755 --- a/configure +++ b/configure @@ -1532,7 +1532,6 @@ EXTERNAL_LIBRARY_LIST="

[FFmpeg-devel] [PATCH 01/16] build: group z libs with other autodetected libraries

2017-08-30 Thread Clément Bœsch
From: Clément Bœsch --- configure | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 61d8160491..44e45791c2 100755 --- a/configure +++ b/configure @@ -3602,6 +3602,9 @@ enable_weak xlib enable_weak cuda cuvid nvenc

Re: [FFmpeg-devel] [PATCH v17] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Timo Rothenpieler
Just wanted to point out, as stated on IRC, that dashenc already has a function that seems to safely solve the filename template issue, and looks like it can be moved into a common file + header, and reused as is:

Re: [FFmpeg-devel] [PATCHv6 4/4] libavcodec: v4l2: add support for v4l2 mem2mem codecs

2017-08-30 Thread Jorge Ramirez
On 08/28/2017 11:52 AM, wm4 wrote: On Sun, 27 Aug 2017 18:26:32 +0200 Jorge Ramirez wrote: On 08/25/2017 05:35 PM, wm4 wrote: That looks generally OK. Is there any chance a hwaccel approach would be possible instead? If I've learned anything about hardware

Re: [FFmpeg-devel] [PATCH] libavformat/dv : read dv audio as BE

2017-08-30 Thread Александр Слободенюк
#include "libavutil/bswap.h" was not necessary for the compilation. > I'm really sorry for the waste of your time... very shameful, that I > didn't mind to run fate first.. > The patch had 2 different bugs at the same time: > - samples were restricted to one byte being taken from

Re: [FFmpeg-devel] [PATCH v17] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Steven Liu
2017-08-30 17:43 GMT+08:00 Timo Rothenpieler : >> v16 fixed: >> 1. remove the snprintf and instead by get_segment_filename make safety >> 2. remove unnecessary loops >> 3. updated xmlStrcmp and xmlFree to av_* functions >> 4. merge code repeat into one function >> 5. add

Re: [FFmpeg-devel] [PATCH v2] fate: add tests for psnr and ssim filter

2017-08-30 Thread Tobias Rapp
On 30.08.2017 12:08, Nicolas George wrote: Le tridi 13 fructidor, an CCXXV, Tobias Rapp a écrit : Metadata output is passed through an Awk script reducing precision of floats to account for architecture differences (e.g x86-32 vs. x86-64). I do not like that approach at all: it will miss

Re: [FFmpeg-devel] [PATCH v10] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread wm4
On Wed, 30 Aug 2017 18:17:45 +0800 Steven Liu wrote: > 2017-08-30 17:35 GMT+08:00 wm4 : > > On Wed, 30 Aug 2017 17:01:40 +0800 > > Steven Liu wrote: > > > >> 2017-08-30 16:18 GMT+08:00 Dan Shamir : > >

Re: [FFmpeg-devel] [PATCH] JPEG200 encoding : added option for changing default codeblock size

2017-08-30 Thread Francesco, Cuzzocrea
Hi Moritz I check the code and it seems the error born from a previous version of ISO/IEC15444-1 I have (I used a final draft not the final document and the definition was quite fuzzy).  Checking final document I found that you are right. According to this specification (Table A-18 Width or

Re: [FFmpeg-devel] [PATCH v10] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Steven Liu
2017-08-30 17:35 GMT+08:00 wm4 : > On Wed, 30 Aug 2017 17:01:40 +0800 > Steven Liu wrote: > >> 2017-08-30 16:18 GMT+08:00 Dan Shamir : > >> What? > > Indeed. Can you people please > > 1. Stop randomly reposting various versions of

Re: [FFmpeg-devel] [PATCH v2] fate: add tests for psnr and ssim filter

2017-08-30 Thread Nicolas George
Le tridi 13 fructidor, an CCXXV, Tobias Rapp a écrit : > Metadata output is passed through an Awk script reducing precision of > floats to account for architecture differences (e.g x86-32 vs. x86-64). I do not like that approach at all: it will miss small differences introduced inadvertently by

[FFmpeg-devel] [PATCH]lavc/hevc_ps: Be less strict on sps conformance by default.

2017-08-30 Thread Carl Eugen Hoyos
Hi! Attached patch allows users to control the sps conformance requirement: It allows to revert the fix for ticket #4035, so faulty streams can be ignored. Also fixes a user-provided sample. Please comment, Carl Eugen From d2a4241f52c7e5b4b5226f7aed237e7448815b77 Mon Sep 17 00:00:00 2001 From:

[FFmpeg-devel] [PATCH v2] fate: add tests for psnr and ssim filter

2017-08-30 Thread Tobias Rapp
Metadata output is passed through an Awk script reducing precision of floats to account for architecture differences (e.g x86-32 vs. x86-64). Signed-off-by: Tobias Rapp --- v2: - removed CPUFLAGS work-around for ssim filter issue - added metadata float value

Re: [FFmpeg-devel] [PATCH] libavformat/dv : read dv audio as BE

2017-08-30 Thread Александр Слободенюк
I'm really sorry for the waste of your time... very shameful, that I didn't mind to run fate first.. The patch had 2 different bugs at the same time: - samples were restricted to one byte being taken from uint8_t pointer - 0x8000 magic was platform-related Sending the fixed version.

Re: [FFmpeg-devel] [PATCH v17] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Timo Rothenpieler
v16 fixed: 1. remove the snprintf and instead by get_segment_filename make safety 2. remove unnecessary loops 3. updated xmlStrcmp and xmlFree to av_* functions 4. merge code repeat into one function 5. add memory alloc faild check 6. update update_init_section and open_url 7. output safety error

Re: [FFmpeg-devel] [PATCH v10] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread wm4
On Wed, 30 Aug 2017 17:01:40 +0800 Steven Liu wrote: > 2017-08-30 16:18 GMT+08:00 Dan Shamir : > What? Indeed. Can you people please 1. Stop randomly reposting various versions of the patch in a chaotic manner, 2. Shorten your mails, and remove

Re: [FFmpeg-devel] [PATCH 07/16] lavfi/framesync2: add common options.

2017-08-30 Thread Paul B Mahol
On 8/10/17, Nicolas George wrote: > Also add functions and macros to help filters chaining these > options to their own. > > Signed-off-by: Nicolas George > --- > libavfilter/framesync2.c | 55 > ++-- >

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mxfenc: Replace literal numbers by named enum values.

2017-08-30 Thread Александр Слободенюк
Well, sorry, that's not about this patch.. > Piece of code: > cid = header_cid[0] << 24 | header_cid[1] << 16 | header_cid[2] << 8 | > header_cid[3]; > .. > switch (cid) { > case 1235: > .. is platform related, isn't it? It seems to me, that it shouldn't work on > BE machine. >> On Tue,

Re: [FFmpeg-devel] [PATCH] compat/cuda/ptx2c: strip CR from each line

2017-08-30 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] configure: add support for libnpp* from cuda sdk 9

2017-08-30 Thread Timo Rothenpieler
applied smime.p7s Description: S/MIME Cryptographic Signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mxfenc: Replace literal numbers by named enum values.

2017-08-30 Thread Александр Слободенюк
Piece of code: cid = header_cid[0] << 24 | header_cid[1] << 16 | header_cid[2] << 8 | header_cid[3]; .. switch (cid) { case 1235: .. is platform related, isn't it? It seems to me, that it shouldn't work on BE machine. > On Tue, Aug 29, 2017 at 02:13:21AM +0200, Michael Niedermayer wrote:

Re: [FFmpeg-devel] [PATCH v10] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Steven Liu
2017-08-30 16:18 GMT+08:00 Dan Shamir : > -ד-דדש--- What? > > On Mar 29, 2017 3:42 PM, "Steven Liu" wrote: > > v2 fixed: > 1. from autodetect to disabled > 2. from camelCase code style to ffmpeg code style > 3. from RepType to AVMediaType > 4. fix variable

Re: [FFmpeg-devel] [PATCH] avfilter/vf_overlay: Restore shorthand option order

2017-08-30 Thread Marton Balint
On Wed, 30 Aug 2017, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer --- libavfilter/framesync2.c | 6 -- libavfilter/framesync2.h | 6 ++ libavfilter/vf_overlay.c | 8 3 files changed, 14 insertions(+), 6 deletions(-) If this is really

Re: [FFmpeg-devel] [Patch] CUDA Thumbnail Filter

2017-08-30 Thread Timo Rothenpieler
Am 30.08.2017 um 06:19 schrieb Yogender Gupta: Attached is a CUDA version of the thumbnail filter, this helps accelerate thumbnails generations significantly, when using the GPU pipeline. Without having done a full review on the code yet: A new filter needs a libavfilter version bump(not 100%

Re: [FFmpeg-devel] [PATCH v10] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Dan Shamir
-ד-דדש--- On Mar 29, 2017 3:42 PM, "Steven Liu" wrote: v2 fixed: 1. from autodetect to disabled 2. from camelCase code style to ffmpeg code style 3. from RepType to AVMediaType 4. fix variable typo 5. change time value from uint32_t to uint64_t 6. removed be used once API

Re: [FFmpeg-devel] [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread 刘歧
> 在 2017年8月30日,16:15,samsamsam 写道: > > Sorry, my fault. Such simple check proposed by me is incorrect. > It not handle basic "%05d" format. That’s no matter samsamsam, if the string is not standard as specification describe, the result maybe need give an error. So that is not

[FFmpeg-devel] [PATCH v19] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread Steven Liu
ffmpeg need a dash demuxer for demux the dash formats base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch TODO: 1. support multi bitrate dash v2 fixed: 1. from autodetect to disabled 2. from camelCase code style to ffmpeg

Re: [FFmpeg-devel] [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread 刘歧
> 在 2017年8月30日,15:52,samsamsam 写道: > > What will happen when $Number%02s%08d$ > > Exacly. Such format is not allowed by dash specification. So, such validation > is sufficient. > > Please check DASH specification format like that $Number%02s%08d$ is not > allowed. > So

Re: [FFmpeg-devel] [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread 刘歧
> 在 2017年8月30日,15:32,刘歧 写道: > > >> 在 2017年8月30日,14:38,samsamsam 写道: >> >> But you do not added this: >> static int get_repl_pattern_and_format(const char *i_url, const char >> *i_marker, char **o_pattern, char **o_format) >> { >> ... >> +

Re: [FFmpeg-devel] [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread 刘歧
> 在 2017年8月30日,14:38,samsamsam 写道: > > But you do not added this: > static int get_repl_pattern_and_format(const char *i_url, const char > *i_marker, char **o_pattern, char **o_format) > { > ... > +for(ptr=start + marker_len; ptr < (end - 1); ++ptr) { /*there is >

[FFmpeg-devel] Odp: Re: [PATCH v15] avformat/dashdec: add dash demuxer base version

2017-08-30 Thread samsamsam
and think about the safety : %02c%lld %s%d%d%d%d What? With my solution this is not problem. Why you think %s%d%d%d%d or %02c%lld give any problem? Dnia 28 sierpnia 2017 12:27 刘歧 l...@chinaffmpeg.org