Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-19 Thread Anton Khirnov
Quoting Diego Biurrun (2016-10-19 10:52:30) > On Tue, Oct 18, 2016 at 09:31:15PM +0200, Anton Khirnov wrote: > > Quoting Diego Biurrun (2016-10-14 01:15:32) > > > On Fri, Oct 14, 2016 at 01:13:42AM +0200, Diego Biurrun wrote: > > > > On Thu, Oct 13, 2016 at 07:03:01PM -0400, Vittorio Giovara

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-19 Thread Diego Biurrun
On Tue, Oct 18, 2016 at 09:31:15PM +0200, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-10-14 01:15:32) > > On Fri, Oct 14, 2016 at 01:13:42AM +0200, Diego Biurrun wrote: > > > On Thu, Oct 13, 2016 at 07:03:01PM -0400, Vittorio Giovara wrote: > > > > --- /dev/null > > > > +++

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-18 Thread Anton Khirnov
Quoting Diego Biurrun (2016-10-14 01:15:32) > On Fri, Oct 14, 2016 at 01:13:42AM +0200, Diego Biurrun wrote: > > On Thu, Oct 13, 2016 at 07:03:01PM -0400, Vittorio Giovara wrote: > > > --- /dev/null > > > +++ b/tests/fate/mov.mak > > > @@ -0,0 +1,12 @@ > > > +FATE_MOV += fate-mov-display-matrix >

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-17 Thread Diego Biurrun
On Mon, Oct 17, 2016 at 05:57:24PM -0400, Vittorio Giovara wrote: > Rotation, aspect ratio and pure matrix export. > > Signed-off-by: Vittorio Giovara > --- > Updated according to Diego's review. > I haven't split the AR tests, I think it makes sense to have it in a

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-17 Thread Diego Biurrun
On Mon, Oct 17, 2016 at 01:47:32PM -0400, Vittorio Giovara wrote: > On Mon, Oct 17, 2016 at 1:30 PM, Diego Biurrun wrote: > > Looking at this in more detail now that the first round of review is over. > > > > On Mon, Oct 17, 2016 at 11:54:04AM -0400, Vittorio Giovara wrote: > >>

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-17 Thread Vittorio Giovara
On Mon, Oct 17, 2016 at 1:30 PM, Diego Biurrun wrote: > Looking at this in more detail now that the first round of review is over. > > On Mon, Oct 17, 2016 at 11:54:04AM -0400, Vittorio Giovara wrote: >> --- a/tests/fate-run.sh >> +++ b/tests/fate-run.sh >> @@ -76,6 +76,11 @@

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-17 Thread Diego Biurrun
Looking at this in more detail now that the first round of review is over. On Mon, Oct 17, 2016 at 11:54:04AM -0400, Vittorio Giovara wrote: > --- a/tests/fate-run.sh > +++ b/tests/fate-run.sh > @@ -76,6 +76,11 @@ probefmt(){ > > +probear(){ > +run avprobe -show_stream_entry

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-13 Thread Diego Biurrun
On Fri, Oct 14, 2016 at 01:13:42AM +0200, Diego Biurrun wrote: > On Thu, Oct 13, 2016 at 07:03:01PM -0400, Vittorio Giovara wrote: > > --- /dev/null > > +++ b/tests/fate/mov.mak > > @@ -0,0 +1,12 @@ > > +FATE_MOV += fate-mov-display-matrix > > +fate-mov-display-matrix: CMD = run avprobe -v 0

Re: [libav-devel] [PATCH] fate: Add tests for mov display matrix

2016-10-13 Thread Diego Biurrun
On Thu, Oct 13, 2016 at 07:03:01PM -0400, Vittorio Giovara wrote: > --- /dev/null > +++ b/tests/fate/mov.mak > @@ -0,0 +1,12 @@ > +FATE_MOV += fate-mov-display-matrix > +fate-mov-display-matrix: CMD = run avprobe -v 0 -show_stream_entry matrix > $(TARGET_SAMPLES)/mov/displaymatrix.mov > + >