Re: [libav-devel] [PATCH 1/3] movenc: Add an option for enabling negative CTS offsets

2017-02-22 Thread Yusuke Nakamura
2017-02-20 6:22 GMT+09:00 Martin Storsjö : > This reduces the need for an edit list; streams that start with > e.g. dts=-1, pts=0 can be encoded as dts=0, pts=0 (which is valid > in mov/mp4) by shifting the dts values of all packets forward. > This avoids the need for edit lists

Re: [libav-devel] [PATCH 2/3] movenc: Don't write any edit list if the start offset is zero

2017-02-22 Thread Yusuke Nakamura
2017-02-22 6:45 GMT+09:00 Martin Storsjö : > Hi Yusuke, > > > On Wed, 22 Feb 2017, Yusuke Nakamura wrote: > > 2017-02-20 6:22 GMT+09:00 Martin Storsjö : >> >> In these cases, the CTTS flag is set, but no edit list is necessary. >>> --- >>> libavformat/movenc.c

Re: [libav-devel] [PATCH 1/3] movenc: Add an option for enabling negative CTS offsets

2017-02-22 Thread Jan Ekstrom
On Sun, Feb 19, 2017 at 11:22 PM, Martin Storsjö wrote: > This reduces the need for an edit list; streams that start with > e.g. dts=-1, pts=0 can be encoded as dts=0, pts=0 (which is valid > in mov/mp4) by shifting the dts values of all packets forward. > This avoids the need

Re: [libav-devel] [PATCH] movenc: factor initial_padding into edit lists

2017-02-22 Thread Martin Storsjö
On Wed, 22 Feb 2017, John Stebbins wrote: initial_padding was getting added to the edit list indirectly due to initial negative dts. But in cases where the audio is delayed, all or part of initial_padding would be unaccounted for. This patch makes initial_padding explicit. ---

[libav-devel] [PATCH 3/3] Add Apple Pixlet decoder

2017-02-22 Thread Vittorio Giovara
From: Paul B Mahol Signed-off-by: Vittorio Giovara --- Changes from last iteration: - lowpass_prediction is rewritten to be clearer (by Anton) - bitstream_unget is used instead of read + branch/skip Missing version.h bump, I'll add it at push time.

[libav-devel] [PATCH 1/3] intmath: add faster clz support

2017-02-22 Thread Vittorio Giovara
From: Ganesh Ajjanagadde --- libavutil/intmath.h | 19 +++ 1 file changed, 19 insertions(+) diff --git a/libavutil/intmath.h b/libavutil/intmath.h index a5ee652..780bbab 100644 --- a/libavutil/intmath.h +++ b/libavutil/intmath.h @@ -44,6 +44,10 @@ #

[libav-devel] [PATCH 2/3] libavutil: add av_mod_uintp2

2017-02-22 Thread Vittorio Giovara
From: James Almer Signed-off-by: James Almer --- libavutil/common.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavutil/common.h b/libavutil/common.h index d2ddcba..3832f10 100644 --- a/libavutil/common.h +++ b/libavutil/common.h

Re: [libav-devel] [PATCH 02/24] hwcontext: Add function to find an AVHWDeviceType by name

2017-02-22 Thread Anton Khirnov
Quoting Mark Thompson (2017-02-19 19:46:24) > --- > libavutil/hwcontext.c | 11 +++ > libavutil/hwcontext.h | 8 > 2 files changed, 19 insertions(+) > > diff --git a/libavutil/hwcontext.c b/libavutil/hwcontext.c > index 1f9442622..e3484c78c 100644 > --- a/libavutil/hwcontext.c

Re: [libav-devel] [PATCH 5/5] fate: Add build-only targets to FATE

2017-02-22 Thread Diego Biurrun
On Wed, Feb 22, 2017 at 06:03:42PM +0100, Diego Biurrun wrote: > On Wed, Feb 22, 2017 at 11:49:36AM -0500, Vittorio Giovara wrote: > > On Wed, Feb 22, 2017 at 11:39 AM, Diego Biurrun wrote: > > > --- /dev/null > > > +++ b/tests/fate/examples.mak > > > @@ -0,0 +1,32 @@ > > >

Re: [libav-devel] [PATCH 5/5] fate: Add build-only targets to FATE

2017-02-22 Thread Diego Biurrun
On Wed, Feb 22, 2017 at 11:49:36AM -0500, Vittorio Giovara wrote: > On Wed, Feb 22, 2017 at 11:39 AM, Diego Biurrun wrote: > > --- /dev/null > > +++ b/tests/fate/examples.mak > > @@ -0,0 +1,32 @@ > > +FATE_EXAMPLES-$(CONFIG_DECODE_AUDIO_EXAMPLE)+= > >

Re: [libav-devel] [PATCH] build: Generalize yasm/nasm-related variable names

2017-02-22 Thread Diego Biurrun
On Mon, Feb 20, 2017 at 11:19:38PM +0200, Martin Storsjö wrote: > On Fri, 17 Feb 2017, Diego Biurrun wrote: > > >None of them are specific to the YASM assembler. > >--- > > > >"x86-asm" it is. > > Kinda ok with me, but I want to hear acks/nacks from others as well. > > For the configure option

Re: [libav-devel] [PATCH 5/5] fate: Add build-only targets to FATE

2017-02-22 Thread Vittorio Giovara
On Wed, Feb 22, 2017 at 11:39 AM, Diego Biurrun wrote: > --- > diff --git a/tests/fate/examples.mak b/tests/fate/examples.mak > new file mode 100644 > index 000..9809645 > --- /dev/null > +++ b/tests/fate/examples.mak > @@ -0,0 +1,32 @@ >

Re: [libav-devel] [PATCH 3/5] fate: Rename WMV8_DRM decoder tests to WMV3_DRM

2017-02-22 Thread Vittorio Giovara
On Wed, Feb 22, 2017 at 11:39 AM, Diego Biurrun wrote: > The codec used in those files is WMV3/WMV9, not WMV2/WMV8. > --- > tests/fate/microsoft.mak | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/tests/fate/microsoft.mak

Re: [libav-devel] [PATCH 2/5] Use modern avconv syntax for codec selection in documentation and tests

2017-02-22 Thread Vittorio Giovara
On Wed, Feb 22, 2017 at 11:39 AM, Diego Biurrun wrote: > --- > doc/encoders.texi| 2 +- > doc/faq.texi | 8 > doc/filters.texi | 4 ++-- > tests/fate-run.sh| 4 ++-- > tests/fate/demux.mak | 34

[libav-devel] [PATCH 2/5] Use modern avconv syntax for codec selection in documentation and tests

2017-02-22 Thread Diego Biurrun
--- doc/encoders.texi| 2 +- doc/faq.texi | 8 doc/filters.texi | 4 ++-- tests/fate-run.sh| 4 ++-- tests/fate/demux.mak | 34 +- tests/fate/h264.mak | 2 +- tests/fate/microsoft.mak | 2 +-

[libav-devel] [PATCH 3/5] fate: Rename WMV8_DRM decoder tests to WMV3_DRM

2017-02-22 Thread Diego Biurrun
The codec used in those files is WMV3/WMV9, not WMV2/WMV8. --- tests/fate/microsoft.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/microsoft.mak b/tests/fate/microsoft.mak index 5b32c7f..4e66307 100644 --- a/tests/fate/microsoft.mak +++

[libav-devel] [PATCH 4/5] fate: Make null comparison method more useful

2017-02-22 Thread Diego Biurrun
This allows dropping /dev/null as reference value when no output is generated. --- tests/fate-run.sh | 2 +- tests/fate/libavcodec.mak | 4 +--- tests/fate/libavutil.mak | 13 ++--- 3 files changed, 8 insertions(+), 11 deletions(-) diff --git a/tests/fate-run.sh

[libav-devel] [PATCH 5/5] fate: Add build-only targets to FATE

2017-02-22 Thread Diego Biurrun
--- People keep forgetting to run "make check", this integrates the build targets into fate so they will show up as failed tests. tests/Makefile | 1 + tests/fate-run.sh | 4 tests/fate/build.mak| 14 ++ tests/fate/examples.mak | 32

[libav-devel] [PATCH 1/5] build: Drop DOC_ prefix from EXAMPLES-related variables

2017-02-22 Thread Diego Biurrun
--- doc/examples/Makefile | 46 +++--- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/doc/examples/Makefile b/doc/examples/Makefile index c42c68f..40c9557 100644 --- a/doc/examples/Makefile +++ b/doc/examples/Makefile @@ -1,31 +1,31 @@

Re: [libav-devel] [PATCH] fate: Add another SVQ3 test to increase coverage

2017-02-22 Thread Diego Biurrun
On Sun, Feb 05, 2017 at 01:14:46PM +0100, Diego Biurrun wrote: > --- > > This covers a regression I introduced a long time ago. > > tests/fate/qt.mak | 10 -- > tests/ref/fate/{svq3 => svq3-1} | 0 > tests/ref/fate/svq3-2 | 20 > 3 files

[libav-devel] [PATCH] movenc: factor initial_padding into edit lists

2017-02-22 Thread John Stebbins
initial_padding was getting added to the edit list indirectly due to initial negative dts. But in cases where the audio is delayed, all or part of initial_padding would be unaccounted for. This patch makes initial_padding explicit. --- libavformat/movenc.c | 60

Re: [libav-devel] [PATCH] movenc: factor initial_padding into edit lists

2017-02-22 Thread John Stebbins
On 02/22/2017 03:41 AM, Martin Storsjö wrote: > On Tue, 21 Feb 2017, John Stebbins wrote: > >> On 02/21/2017 02:54 PM, Martin Storsjö wrote: >>> On Tue, 21 Feb 2017, John Stebbins wrote: >>> initial_padding was getting added to the edit list indirectly due to initial negative dts. But

Re: [libav-devel] [PATCH 1/2] Add Cineform HD Decoder

2017-02-22 Thread Luca Barbato
On 22/02/2017 13:44, Diego Biurrun wrote: > From: Kieran Kunhya > > Decodes YUV 4:2:2 10-bit and RGB 12-bit files. > Older files with more subbands, skips, Bayer, alpha not supported. > > Further fixes and refactorings by Anton Khirnov , > Diego Biurrun

Re: [libav-devel] [PATCH 2/2] cfhd: Add FATE tests

2017-02-22 Thread Luca Barbato
On 22/02/2017 13:44, Diego Biurrun wrote: > --- > tests/fate/video.mak | 12 > tests/ref/fate/cfhd-1 | 11 +++ > tests/ref/fate/cfhd-2 | 11 +++ > tests/ref/fate/cfhd-3 | 11 +++ > 4 files changed, 45 insertions(+) > create mode 100644 tests/ref/fate/cfhd-1

[libav-devel] [PATCH 1/2] Add Cineform HD Decoder

2017-02-22 Thread Diego Biurrun
From: Kieran Kunhya Decodes YUV 4:2:2 10-bit and RGB 12-bit files. Older files with more subbands, skips, Bayer, alpha not supported. Further fixes and refactorings by Anton Khirnov , Diego Biurrun , Vittorio Giovara

[libav-devel] [PATCH 2/2] cfhd: Add FATE tests

2017-02-22 Thread Diego Biurrun
--- tests/fate/video.mak | 12 tests/ref/fate/cfhd-1 | 11 +++ tests/ref/fate/cfhd-2 | 11 +++ tests/ref/fate/cfhd-3 | 11 +++ 4 files changed, 45 insertions(+) create mode 100644 tests/ref/fate/cfhd-1 create mode 100644 tests/ref/fate/cfhd-2 create mode

Re: [libav-devel] [PATCH] Place attribute_deprecated in the right position for struct declarations

2017-02-22 Thread Mark Thompson
On 22/02/17 10:59, Diego Biurrun wrote: > libavcodec/vaapi.h:58:1: warning: attribute 'deprecated' is ignored, place it > after "struct" to apply attribute to type declaration [-Wignored-attributes] > --- > libavcodec/vaapi.h | 3 +-- > libavcodec/xvmc.h | 2 +- > 2 files changed, 2

[libav-devel] [PATCH] Place attribute_deprecated in the right position for struct declarations

2017-02-22 Thread Diego Biurrun
libavcodec/vaapi.h:58:1: warning: attribute 'deprecated' is ignored, place it after "struct" to apply attribute to type declaration [-Wignored-attributes] --- libavcodec/vaapi.h | 3 +-- libavcodec/xvmc.h | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/vaapi.h

Re: [libav-devel] [PATCH] movenc: factor initial_padding into edit lists

2017-02-22 Thread Martin Storsjö
On Tue, 21 Feb 2017, John Stebbins wrote: On 02/21/2017 02:54 PM, Martin Storsjö wrote: On Tue, 21 Feb 2017, John Stebbins wrote: initial_padding was getting added to the edit list indirectly due to initial negative dts. But in cases where the audio is delayed, all or part of

Re: [libav-devel] [PATCH] mkv: Update the seek test to match 5d3953a5dc

2017-02-22 Thread Luca Barbato
On 22/02/2017 10:11, Diego Biurrun wrote: > On Wed, Feb 22, 2017 at 09:56:36AM +0100, Luca Barbato wrote: >> --- >> tests/ref/seek/lavf-mkv | 26 +- >> 1 file changed, 13 insertions(+), 13 deletions(-) > > Hopefully OK > fate seems happy with this :)

Re: [libav-devel] [PATCH] mkv: Update the seek test to match 5d3953a5dc

2017-02-22 Thread Diego Biurrun
On Wed, Feb 22, 2017 at 09:56:36AM +0100, Luca Barbato wrote: > --- > tests/ref/seek/lavf-mkv | 26 +- > 1 file changed, 13 insertions(+), 13 deletions(-) Hopefully OK Diego ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH] mkv: Update the seek test to match 5d3953a5dc

2017-02-22 Thread Luca Barbato
--- tests/ref/seek/lavf-mkv | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/tests/ref/seek/lavf-mkv b/tests/ref/seek/lavf-mkv index 530c8df..8267a2a 100644 --- a/tests/ref/seek/lavf-mkv +++ b/tests/ref/seek/lavf-mkv @@ -1,53 +1,53 @@ -ret: 0

Re: [libav-devel] [PATCH] fate: Update fate-lavf-mkv after commit 5d3953a5dc

2017-02-22 Thread Luca Barbato
On 22/02/2017 06:53, wm4 wrote: > Come on, either it's ok or it's not ok. I was wondering if it was enough, in fact it is not. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 3/3] build: Fine-grained link-time dependency settings

2017-02-22 Thread Diego Biurrun
On Tue, Feb 21, 2017 at 07:58:04PM +0100, Luca Barbato wrote: > On 21/02/2017 18:26, Diego Biurrun wrote: > > all: $(AVPROGS) alltools checkheaders examples testprogs > > What? That's a local change that I have to force building all targets on Oracle. Notice that it's not part of the patch