[libav-devel] [PATCH 2/4] lavu: Add AVSphericalMapping type and frame side data

While no decoder currently exports spherical information, this type represents a frame property that has to be passed through from container to frames. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Applied the necessary doxygen changes, still undecided on the `double`

[libav-devel] [PATCH 4/4] mov: Export spherical information

This implements Spherical Video V1 and V2, as described in the spatial-media collection by Google. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- No changes. Vittorio Changelog | 1 + libavformat/isom.h | 6 ++ libavformat/mov.c

[libav-devel] [PATCH 3/4] lavc: Add spherical packet side data API

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- No changes. avprobe.c| 28 doc/APIchanges | 4 libavcodec/avcodec.h | 6 ++ libavcodec/utils.c | 1 + libavcodec/version.h | 4 ++-- libavformat/dump.c

[libav-devel] [PATCH 1/4] lavu: Add a video section to general documentation

Fill it with AVStereo3D and AVDisplayMatrix documentation. Apply the necessary changes to make verbatim code look good in doxygen. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavutil/avutil.h | 6 ++ libavutil/display.h | 27 +++ lib

Re: [libav-devel] [PATCH 13/13] examples/decode_video: allocate the packet dynamically

On Thu, Nov 10, 2016 at 10:51 AM, Anton Khirnov wrote: > AVPackets on stack are discouraged. > --- > doc/examples/decode_video.c | 13 - > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/doc/examples/decode_video.c b/doc/examples/decode_video.c >

Re: [libav-devel] [PATCH 10/13] examples/encode_video: allocate the packet dynamically

On Thu, Nov 10, 2016 at 10:51 AM, Anton Khirnov wrote: > AVPackets on stack are discouraged. > --- > doc/examples/encode_video.c | 15 --- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/doc/examples/encode_video.c b/doc/examples/encode_video.c >

Re: [libav-devel] [PATCH 07/13] examples/decode_audio: allocate the packet dynamically

On Thu, Nov 10, 2016 at 10:51 AM, Anton Khirnov wrote: > AVPackets on stack are discouraged now. > --- > doc/examples/decode_audio.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/doc/examples/decode_audio.c b/doc/examples/decode_audio.c >

Re: [libav-devel] [PATCH] libxvid: Require availability of mkstemp()

On Thu, Nov 10, 2016 at 11:29 AM, Diego Biurrun wrote: > The replacement code uses tempnam(), which is dangerous. > Such a fringe feature is not worth the trouble. > --- > > Now without stray changes to the build system. > > configure | 2 +- >

[libav-devel] [PATCH] fate: Add h264 and hevc extradata reload tests

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- tests/fate/h264.mak | 5 + tests/fate/hevc.mak | 5 + tests/ref/fate/h264-extradata-reload | 5 + tests/ref/fate/hevc-extradata-reload | 5 + 4 files changed, 20 inse

Re: [libav-devel] [PATCH 7/7] configure: Move license checks directly after command line parsing

On Tue, Nov 8, 2016 at 2:35 PM, Diego Biurrun wrote: > This will allow to error out immediately if incompatible options are > passed on the command line instead of running time-consuming tests. > --- > configure | 21 +++-- > 1 file changed, 11 insertions(+), 10

Re: [libav-devel] [PATCH 6/7] configure: Log name and parameters of all helper functions where it makes sense

On Tue, Nov 8, 2016 at 2:35 PM, Diego Biurrun wrote: > --- > configure | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/configure b/configure > index 422036b..70a0d95 100755 > --- a/configure > +++ b/configure > @@ -1027,6 +1027,7 @@ check_exec(){ > } > >

Re: [libav-devel] [PATCH 5/7] configure: Use check_cpp in CPP flags tests

On Tue, Nov 8, 2016 at 2:35 PM, Diego Biurrun wrote: > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index 4cfa066..422036b 100755 > --- a/configure > +++ b/configure > @@ -865,7 +865,7 @@ check_code(){ > >

Re: [libav-devel] [PATCH 2/7] configure: Print list of enabled programs

On Tue, Nov 8, 2016 at 2:35 PM, Diego Biurrun wrote: > Also drop a related and now redundant informative output line. > --- > configure | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 7dda871..a6617b5 100755 > ---

Re: [libav-devel] [PATCH 1/7] configure: Improve output wording

On Tue, Nov 8, 2016 at 2:35 PM, Diego Biurrun wrote: > Also drop a redundant output line. > --- > configure | 9 - > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/configure b/configure > index e862757..7dda871 100755 > --- a/configure > +++

Re: [libav-devel] [RFC] avpacket: Mark src pointer as constant

On Tue, Nov 8, 2016 at 1:35 PM, Luca Barbato wrote: > --- > > libavcodec/avcodec.h | 2 +- > libavcodec/avpacket.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 88e6c62..a770cf2 100644 > ---

Re: [libav-devel] [PATCH] avconv: Drop stray leftover debug output

On Tue, Nov 8, 2016 at 11:49 AM, Diego Biurrun wrote: > --- > avconv.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/avconv.c b/avconv.c > index 769b9e0..97166d9 100644 > --- a/avconv.c > +++ b/avconv.c > @@ -206,7 +206,6 @@ static void avconv_cleanup(int ret) >

Re: [libav-devel] [PATCH 2/2] hevc: Support extradata changes

On Mon, Nov 7, 2016 at 5:44 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> > --- > This version avoids fiddling with freeing/allocating context extradata. > I'm not a 100% sure it is needed, but I rec

Re: [libav-devel] [PATCH] mov: Read multiple stsd from DV

On Mon, Nov 7, 2016 at 10:30 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> > --- > libavformat/mov.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/mov.c b/libavformat/

[libav-devel] [PATCH] mov: Read multiple stsd from DV

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavformat/mov.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index e9318bd..475a9ef 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1785,6 +1785,8 @@ stat

[libav-devel] [PATCH 1/2] hevc: Allow parsing external extradata buffers

--- Necessary for the following commit. It mimics what is done for h264. Vittorio libavcodec/hevcdec.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c index 4dacdd7..d946337 100644 --- a/libavcodec/hevcdec.c +++

[libav-devel] [PATCH 2/2] hevc: Support extradata changes

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- This version avoids fiddling with freeing/allocating context extradata. I'm not a 100% sure it is needed, but I recall having some problems when reallocating the extradata buffer from context and frame threading for h264. Vi

Re: [libav-devel] [PATCH 1/3] lavu: Add AVSphericalMapping type and frame side data

On Mon, Nov 7, 2016 at 4:14 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > While no decoder currently exports spherical information, this type > represents a frame property that has to be passed through from container > to frames. > > Signed-off-by: Vittorio G

[libav-devel] [PATCH 2/3] lavc: Add spherical packet side data API

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- No particular change. Vittorio avprobe.c| 28 doc/APIchanges | 4 libavcodec/avcodec.h | 6 ++ libavcodec/utils.c | 1 + libavcodec/version.h | 4 ++-- libav

[libav-devel] [PATCH 3/3] mov: Export spherical information

This implements Spherical Video V1 and V2, as described in the spatial-media collection by Google. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Applied review comments from reviewers (except side data refactoring which may come later). Vittorio libavformat/isom.h

[libav-devel] [PATCH 1/3] lavu: Add AVSphericalMapping type and frame side data

While no decoder currently exports spherical information, this type represents a frame property that has to be passed through from container to frames. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Modified the name, added more documentation, added cropping (!). Moved t

Re: [libav-devel] [PATCH 2/2] swscale: Add GRAY12

On Thu, Nov 3, 2016 at 4:45 PM, Luca Barbato wrote: > --- > libswscale/input.c | 2 ++ > libswscale/swscale_internal.h | 2 ++ > libswscale/swscale_unscaled.c | 1 + > libswscale/utils.c | 2 ++ >

Re: [libav-devel] [PATCH] Drop unreachable break and return statements

On Thu, Nov 3, 2016 at 4:12 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Nov 03, 2016 at 04:04:54PM -0400, Vittorio Giovara wrote: >> On Thu, Nov 3, 2016 at 1:25 PM, Diego Biurrun <di...@biurrun.de> wrote: >> > --- >> > >> > Dropped the

Re: [libav-devel] [PATCH 2/3] avconv_opt: Rework hwaccels array iteration to avoid a warning on empty array

On Thu, Nov 3, 2016 at 3:42 PM, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-11-03 18:32:35) >> avconv_opt.c:188:19: warning: comparison of unsigned expression < 0 is >> always false [-Wtype-limits] >> --- >> avconv_opt.c | 5 ++--- >> 1 file changed, 2 insertions(+), 3

Re: [libav-devel] [PATCH] Drop unreachable break and return statements

On Thu, Nov 3, 2016 at 1:25 PM, Diego Biurrun wrote: > --- > > Dropped the cmdutils.c part, which is a tad more complicated, as noticed by > Luca. > > libavcodec/bmvvideo.c | 1 - > libavcodec/hevc_sei.c | 1 - > libavcodec/indeo3.c| 2 -- > libavcodec/sanm.c | 3

Re: [libav-devel] [PATCH 2/2] hevc: Support extradata changes

On Wed, Nov 2, 2016 at 12:30 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 02/11/2016 17:03, Vittorio Giovara wrote: >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> >> --- >> libavcodec/hevcdec.c | 18 ++ >> libavformat

[libav-devel] [PATCH 1/2] lavc: Add spherical packet side data API

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- avprobe.c| 20 ++ doc/APIchanges | 4 +++ libavcodec/avcodec.h | 74 libavcodec/utils.c | 12 + libavcodec/version.h | 4 +-- libav

[libav-devel] [PATCH 2/2] mov: Export spherical information

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavformat/isom.h | 4 + libavformat/mov.c | 277 + 2 files changed, 281 insertions(+) diff --git a/libavformat/isom.h b/libavformat/isom.h index 58f0a20..2e22575

[libav-devel] [PATCH] lavc: Add hevc main10 profile to avconv cli

--- libavcodec/options_table.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index 995906b..4323ae9 100644 --- a/libavcodec/options_table.h +++ b/libavcodec/options_table.h @@ -394,6 +394,7 @@ static const AVOption avcodec_options[] =

[libav-devel] [PATCH 1/2] hevc: Move hevc_decode_extradata before frame decoding

Avoids a forward-declaration in the following commit. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/hevcdec.c | 148 +-- 1 file changed, 74 insertions(+), 74 deletions(-) diff --git a/libavcodec/hevcdec.c b/liba

[libav-devel] [PATCH 2/2] hevc: Support extradata changes

Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/hevcdec.c | 18 ++ libavformat/mov.c| 12 +--- 2 files changed, 23 insertions(+), 7 deletions(-) diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c index 4dacdd7..44b6108

[libav-devel] [PATCH] mov: Detect spherical videos with packet side data

This includes both versions of the spec, XML and mp4 box. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- avprobe.c| 20 libavcodec/avcodec.h | 44 libavformat/dump.c | 28 ++ libavformat/isom.h | 4 + libavformat/mov.c

Re: [libav-devel] [PATCH 2/2] aactab: Move extern keyword to the front of array declarations

On Mon, Oct 31, 2016 at 5:26 PM, Diego Biurrun wrote: > libavcodec/aactab.h:49:1: warning: ‘extern’ is not at beginning of > declaration [-Wold-style-declaration] > libavcodec/aactab.h:50:1: warning: ‘extern’ is not at beginning of > declaration [-Wold-style-declaration] > ---

Re: [libav-devel] [PATCH 1/2] vf_drawtext: Move static keyword to beginning of variable declaration

On Mon, Oct 31, 2016 at 5:26 PM, Diego Biurrun wrote: > libavfilter/vf_drawtext.c:226:1: warning: ‘static’ is not at beginning of > declaration [-Wold-style-declaration] > --- > libavfilter/vf_drawtext.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff

[libav-devel] [PATCH] lavu: Rename ycgco color space appropriately

Planes are ordered as the name suggests now. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/options_table.h | 3 ++- libavutil/pixdesc.c| 2 +- libavutil/pixfmt.h | 3 ++- libavutil/version.h| 2 +- 4 files changed, 6 insertions

Re: [libav-devel] [PATCH] xcb: Add all the libraries to the link line explicitly

On Sun, Oct 30, 2016 at 12:46 PM, Luca Barbato wrote: > Avoid an underlink issue on recent distributions. An "underlink"? You mean linking less libs than necessary? > CC: libav-sta...@libav.org > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >

Re: [libav-devel] [PATCH 0/3] AviSynth+ pix_fmt updates

On Fri, Oct 28, 2016 at 10:20 PM, Stephen Hutchinson <qyo...@gmail.com> wrote: > On 10/27/2016 11:37 PM, Vittorio Giovara wrote: >> >> yeah 64 bits formats work only in pass-through, so you cannot convert >> them or output anything in them >> > > The ou

Re: [libav-devel] [PATCH] omx: Add HEVC support

On Sat, Aug 20, 2016 at 10:12 AM, Luca Barbato wrote: > --- > configure | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/omx.c | 24 > 3 files changed, 26 insertions(+) probably ok -- Vittorio

Re: [libav-devel] [PATCH 0/3] AviSynth+ pix_fmt updates

Thanks for the set On Wed, Oct 26, 2016 at 10:54 PM, Stephen Hutchinson wrote: > The patch to support AviSynth+'s new pixfmts has been > updated to include GBR(A)P12 and provide correct output > for Planar RGB. > > As previously noted, this patchset assumes the presence > of

Re: [libav-devel] [PATCH] mov: Evaluate the movie display matrix

On Mon, Oct 24, 2016 at 5:46 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. So store it

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

On Tue, Oct 25, 2016 at 7:21 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Oct 24, 2016 at 05:22:55PM -0400, Vittorio Giovara wrote: >> --- /dev/null >> +++ b/tests/fate/mov.mak >> @@ -0,0 +1,15 @@ >> +FATE_MOV += fate-mov-dar >> +fate-mov-dar:

Re: [libav-devel] [PATCH 12/24] examples/decode_audio: allocate the packet dynamically

On Thu, Oct 20, 2016 at 10:07 AM, Anton Khirnov wrote: > AVPackets on stack are discouraged now. > --- > doc/examples/decode_audio.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) seems ok -- Vittorio ___

Re: [libav-devel] [PATCH 19/24] examples/encode_video: allocate the packet dynamically

On Thu, Oct 20, 2016 at 10:07 AM, Anton Khirnov wrote: > AVPackets on stack are discouraged. > --- > doc/examples/encode_video.c | 15 --- > 1 file changed, 8 insertions(+), 7 deletions(-) > > diff --git a/doc/examples/encode_video.c b/doc/examples/encode_video.c >

Re: [libav-devel] [PATCH 13/24] examples/decode_audio: flush the decoder

On Thu, Oct 20, 2016 at 10:07 AM, Anton Khirnov wrote: > --- > doc/examples/decode_audio.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/examples/decode_audio.c b/doc/examples/decode_audio.c > index 8164cfc..454a8fe 100644 > --- a/doc/examples/decode_audio.c

Re: [libav-devel] [PATCH 16/24] examples/encode_video: set the framerate

On Thu, Oct 20, 2016 at 10:07 AM, Anton Khirnov wrote: > --- > doc/examples/encode_video.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/doc/examples/encode_video.c b/doc/examples/encode_video.c > index 72020e0..bd10d40 100644 > ---

Re: [libav-devel] [PATCH 17/24] examples/encode_video: use the AVFrame API for allocating the frame

On Thu, Oct 20, 2016 at 10:07 AM, Anton Khirnov wrote: > It is more efficient and so preferred over allocating the buffers > manually. > --- > doc/examples/encode_video.c | 19 --- > 1 file changed, 12 insertions(+), 7 deletions(-) > > diff --git

[libav-devel] [PATCH] mov: Evaluate the movie display matrix

accordingly. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Now with the correct variable declaration. Forgot to mention the macro is switched to check for identity rather than fullness. Vittorio libavformat/isom.h| 2 ++ libavformat/mov.c

[libav-devel] [PATCH 2/3] fate: Add tests for mov display matrix

Rotation, sample/display aspect ratio and pure matrix export. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Split tests as requested, moved them back to mov.mak. Vittorio tests/Makefile| 1 + tests/fate-run.sh | 4 tests/fate/m

[libav-devel] [PATCH 1/3] avprobe: Add -show_stream_entry to get a single stream property

This is needed for improved fate testing and it is modeled after -show_format_entry. The main behavioral difference is that when a print function is called with an empty key, rather than discarding it, the closes key in the hierarchy is used instead. Signed-off-by: Vittorio Giovara <vittorio.g

[libav-devel] [PATCH 3/3] mov: Evaluate the movie display matrix

accordingly. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Avoid conversion to double. Vittorio libavformat/isom.h| 2 ++ libavformat/mov.c | 48 --- tests/ref/fate/mov-dar| 2 +- tests/ref/fa

Re: [libav-devel] [PATCH 05/24] examples/encode_audio: use the AVFrame API for allocating the data

On Thu, Oct 20, 2016 at 10:07 AM, Anton Khirnov wrote: > It is simpler and more efficient. > --- > doc/examples/encode_audio.c | 28 +++- > 1 file changed, 11 insertions(+), 17 deletions(-) > > diff --git a/doc/examples/encode_audio.c

[libav-devel] [PATCH] mov: Update colr values

For 'nclx', the latest edition of the standard switched from JPEG XR to 23001-8, which matches the current order of our entries. Bound checking is preserved as a sanity check. For 'nclc', qtff edition 2016-09-13 introduced a few new entries. Signed-off-by: Vittorio Giovara <vittorio.g

[libav-devel] [PATCH 2/2] mov: Update colr values

For 'nclx', the latest edition of the standard switched from JPEG XR to 23001-8, which matches the current order of our entries. For 'nclc', qtff edition 2016-09-13 introduced a few new entries. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavformat/mov.

[libav-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

--- libavcodec/options_table.h | 1 + libavutil/pixdesc.c| 1 + libavutil/pixfmt.h | 1 + 3 files changed, 3 insertions(+) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index 5090806..04cb20d 100644 --- a/libavcodec/options_table.h +++

Re: [libav-devel] [PATCH 1/3] avprobe: Add -show_stream_entry to get a single stream property

On Tue, Oct 18, 2016 at 3:38 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-10-14 00:23:08) >> This is needed for improved fate testing and it is modeled after >> -show_format_entry. The main behavioral difference is that when a print

Re: [libav-devel] [PATCH 2/2] hevc: factor out a repeated condition

On Tue, Oct 18, 2016 at 3:24 PM, Anton Khirnov wrote: > --- > libavcodec/hevcdec.c | 20 > 1 file changed, 8 insertions(+), 12 deletions(-) ok i think -- Vittorio ___ libav-devel mailing list

Re: [libav-devel] [PATCH 1/2] hevc: move the SliceType enum to hevc.h

On Tue, Oct 18, 2016 at 3:24 PM, Anton Khirnov wrote: > Those values are decoder-independent and are also use by the VA-API > encoder. > --- possibly ok -- Vittorio ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] fate: Update hashes

How about fate: Update gbrap12le hash Missing from be9dba5c8abc6ecf0b8ee4ccb11c7850327fcf8d. On Tue, Oct 18, 2016 at 9:00 AM, Luca Barbato wrote: > The previous commit lacked this. > --- > > Looks like the hashes weren't updated... > > tests/ref/fate/filter-pixfmts-scale

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

Rotation, aspect ratio and pure matrix export. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Updated according to Diego's review. I haven't split the AR tests, I think it makes sense to have it in a single one, but if people strongly prefer the split version, I'll

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

On Mon, Oct 17, 2016 at 1:30 PM, Diego Biurrun <di...@biurrun.de> 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/fat

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

Rotation, aspect ratio and pure matrix export. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- tests/fate-run.sh | 5 + tests/fate/probe.mak | 13 + tests/ref/fate/mov-ar | 2 ++ tests/ref/fate/mov-display-matri

Re: [libav-devel] [PATCH 3/3] mov: Evaluate the movie display matrix

On Thu, Oct 13, 2016 at 6:26 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > On Thu, Oct 13, 2016 at 6:23 PM, Vittorio Giovara > <vittorio.giov...@gmail.com> wrote: >> This matrix needs to be applied after all others have (currently only >> display

Re: [libav-devel] [PATCH 1/3] avprobe: Add -show_stream_entry to get a single stream property

On Thu, Oct 13, 2016 at 6:23 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > This is needed for improved fate testing and it is modeled after > -show_format_entry. The main behavioral difference is that when a print > function is called with an empty key, rather

Re: [libav-devel] [PATCH] swscale: Properly load alpha for planar rgb

On Sun, Oct 16, 2016 at 2:07 PM, Sean McGovern <gsean...@gmail.com> wrote: > Hi, > > On Oct 14, 2016 17:28, "Luca Barbato" <lu_z...@gentoo.org> wrote: >> >> On 14/10/2016 23:25, Vittorio Giovara wrote: >> > From: Michael Niedermayer <mich

Re: [libav-devel] [PATCH 12/12] x86/yadif-10: remove duplicate ABS macro

On Sun, Oct 16, 2016 at 2:11 PM, Janne Grunau wrote: > From: James Almer > > And use the x86util ones instead, which are optimized for mmxext/sse2. > About ~1% increase in performance on pre SSSE3 processors. Question unrelated to this patch, is there

[libav-devel] [PATCH] swscale: Properly load alpha for planar rgb

From: Michael Niedermayer <michae...@gmx.at> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- This should fix ppc and sun fate tests. Vittorio libswscale/input.c | 37 + tests/ref/fate/filter-pixfmts-scale | 4 +

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

Rotation, aspect ratio and pure matrix export. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- tests/Makefile| 1 + tests/fate-run.sh | 5 + tests/fate/mov.mak| 12 tests/ref/fate/mov-ar

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

On Thu, Oct 13, 2016 at 6:52 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Oct 13, 2016 at 06:23:09PM -0400, Vittorio Giovara wrote: >> Rotation, aspect ratio and pure matrix export. >> >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.

Re: [libav-devel] [PATCH 3/3] mov: Evaluate the movie display matrix

On Thu, Oct 13, 2016 at 6:23 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. > > So sto

[libav-devel] [PATCH 2/3] fate: Add tests for mov display matrix

Rotation, aspect ratio and pure matrix export. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- tests/Makefile| 1 + tests/fate-run.sh | 5 + tests/ref/fate/mov-ar | 2 ++ tests/ref/fate/mov-display-matrix | 9 +

[libav-devel] [PATCH 1/3] avprobe: Add -show_stream_entry to get a single stream property

This is needed for improved fate testing and it is modeled after -show_format_entry. The main behavioral difference is that when a print function is called with an empty key, rather than discarding it, the closes key in the hierarchy is used instead. Signed-off-by: Vittorio Giovara <vittorio.g

[libav-devel] [PATCH 3/3] mov: Evaluate the movie display matrix

is identity and when it is not. Fate tests are updated accordingly. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavformat/isom.h| 2 ++ libavformat/mov.c | 57 +-- tests/ref/fate/mov-ar | 4 +--

Re: [libav-devel] [PATCH] swscale: Add the GBRAP12 output

On Wed, Oct 12, 2016 at 3:32 PM, Luca Barbato wrote: > --- seems ok, maybe just `swscale: Add GBRAP12 output support` -- Vittorio ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] allow to pass custom sidedata through decoders

On Wed, Oct 12, 2016 at 11:43 AM, Francois Cartegnie <fcvlc...@free.fr> wrote: > Le 12/10/2016 à 16:50, Vittorio Giovara a écrit : >> On Wed, Oct 12, 2016 at 8:50 AM, Francois Cartegnie <fcvlc...@free.fr> wrote: >> >> I'm not really sure about this. >> If i

Re: [libav-devel] [PATCH 1/4] Add GBRP12 pixel format support

On Mon, Oct 10, 2016 at 4:35 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > From: Michael Niedermayer <michae...@gmx.at> > > Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> > --- > doc/APIchanges | 3 +++ > libavutil/pixdesc.c | 25 ++

Re: [libav-devel] [PATCH] APIchanges: Expand the name of recently added pixel formats

On Thu, Oct 6, 2016 at 6:29 PM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 07/10/16 00:27, Vittorio Giovara wrote: >> This makes them easier to search for. >> --- >> doc/APIchanges | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >>

Re: [libav-devel] [PATCH] lavc: Document BSF in/out codecparam alloc and init process

On Mon, Oct 10, 2016 at 1:38 PM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-10-04 17:59:53) >> --- >> libavcodec/avcodec.h | 7 +-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/avcodec.h

Re: [libav-devel] [PATCH 2/9] hwcontext_dxva2: add support for the P8 format

On Wed, Oct 12, 2016 at 3:58 AM, Anton Khirnov wrote: > Quoting Hendrik Leppkes (2016-10-11 22:33:43) >> On Tue, Oct 11, 2016 at 9:34 PM, Anton Khirnov wrote: >> > This format is used internally by the QSV encoder to store the encoded >> > bitstream. >> >>

Re: [libav-devel] [PATCH] allow to pass custom sidedata through decoders

On Wed, Oct 12, 2016 at 8:50 AM, Francois Cartegnie wrote: > Because not every side data is handled or extracted > internally by libav, a libavcodec user might need > to attach some extradata and retrieve it in > presentation order after decoding. > Ex: active format data, >

[libav-devel] [PATCH] swscale: Add missing yuv444p12 swapping

Missing from 9bd6ea5695660529b2887292874a7b9e61fc301e. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Fixes the ppc fate tests. Vittorio libswscale/input.c | 4 tests/ref/fate/filter-pixfmts-scale | 2 +- 2 files changed, 5 insertions(+), 1 de

[libav-devel] [PATCH] APIchanges: Expand the name of recently added pixel formats

This makes them easier to search for. --- doc/APIchanges | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 655783e..ef25a98 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -20,7 +20,7 @@ API changes, most recent first: Add

[libav-devel] [PATCH] mov: Evaluate the movie display matrix

is identity and when it is not. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavformat/isom.h | 2 ++ libavformat/mov.c | 64 -- 2 files changed, 55 insertions(+), 11 deletions(-) diff --git a/libavformat/isom.h b/libav

[libav-devel] [PATCH 2/2] mov: Evaluate the movie display matrix

is identity and when it is not. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavformat/isom.h | 2 ++ libavformat/mov.c | 61 +- 2 files changed, 53 insertions(+), 10 deletions(-) diff --git a/libavformat/isom.h b/libav

[libav-devel] [PATCH 1/2] mov: Rewrite aspect ratio handling from display matrix

From: Michael Niedermayer <michae...@gmx.at> This should make this code more robust and simpler. The use of hypot is by Ganesh Ajjanagadde. Also use the context display matrix instead of the local one. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavforma

[libav-devel] [PATCH] lavc: Document BSF in/out codecparam alloc and init process

--- libavcodec/avcodec.h | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 167525d..02f391c 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -5017,12 +5017,15 @@ typedef struct AVBSFContext { void

Re: [libav-devel] [PATCH] lavc: Document in/out AVCodecParameters ownership in AVBSFContext

On Tue, Oct 4, 2016 at 11:50 AM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > On Tue, Oct 4, 2016 at 5:45 PM, Vittorio Giovara > <vittorio.giov...@gmail.com> wrote: >> --- >> libavcodec/avcodec.h | 6 -- >> 1 file changed, 4 insertions(+), 2 deleti

[libav-devel] [PATCH] lavc: Document in/out AVCodecParameters ownership in AVBSFContext

--- libavcodec/avcodec.h | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 167525d..a59d7e9 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -5017,12 +5017,14 @@ typedef struct AVBSFContext { void

Re: [libav-devel] [PATCH 12/15] qdrw: Change type of array stride parameters to ptrdiff_t

On Tue, Sep 6, 2016 at 12:11 PM, Diego Biurrun wrote: > ptrdiff_t is the correct type for array strides and similar. > --- > libavcodec/qdrw.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/qdrw.c b/libavcodec/qdrw.c > index b7493e4..3605a5a

Re: [libav-devel] [PATCH 06/15] dxv: Change type of array stride parameters to ptrdiff_t

On Tue, Sep 6, 2016 at 12:11 PM, Diego Biurrun wrote: > ptrdiff_t is the correct type for array strides and similar. > --- > libavcodec/dxv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/dxv.c b/libavcodec/dxv.c > index 99327df..fce7f62

Re: [libav-devel] [PATCH 13/15] hq_hqa: Change type of array stride parameters to ptrdiff_t

On Tue, Sep 6, 2016 at 12:11 PM, Diego Biurrun wrote: > ptrdiff_t is the correct type for array strides and similar. > --- > libavcodec/hq_hqadsp.c | 3 ++- > libavcodec/hq_hqadsp.h | 3 ++- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git

[libav-devel] [PATCH 1/2] avidec: Do not special case palette on big-endian

This simplifies the code a bit, does not change output data in any way. --- libavformat/avidec.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavformat/avidec.c b/libavformat/avidec.c index 3666b57..e8a19f5 100644 --- a/libavformat/avidec.c +++ b/libavformat/avidec.c

[libav-devel] [PATCH 2/2] avidec: Set palette alpha as fully opaque

Palette format is always in RGBA. --- libavformat/avidec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/avidec.c b/libavformat/avidec.c index e8a19f5..92ff428 100644 --- a/libavformat/avidec.c +++ b/libavformat/avidec.c @@ -653,7 +653,7 @@ static int

Re: [libav-devel] [PATCH] vaapi_encode: Write sequence header as extradata

On Sun, Oct 2, 2016 at 4:19 AM, Mark Thompson wrote: > Only works if packed headers are supported, where we can know the > output before generating the first frame. > --- > If this is wanted when packed headers aren't supported, we will need a > different approach because we

Re: [libav-devel] [PATCH] avidec: Do not special case palette on big-endian

On Sat, Oct 1, 2016 at 5:35 PM, Martin Storsjö <mar...@martin.st> wrote: > On Sat, 1 Oct 2016, Vittorio Giovara wrote: > >> On Wed, Sep 28, 2016 at 2:01 AM, Anton Khirnov <an...@khirnov.net> wrote: >>> >>> Quoting Vittorio Giovara (2016-09-28 00:12:00) &g

Re: [libav-devel] [PATCH] avidec: Do not special case palette on big-endian

On Wed, Sep 28, 2016 at 2:01 AM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-09-28 00:12:00) >> --- >> libavformat/avidec.c | 6 +- >> 1 file changed, 1 insertion(+), 5 deletions(-) >> >> diff --git a/libavformat/avidec.c

Re: [libav-devel] [PATCH 1/2] avconv: make sure the filtergraph is freed on init failure

On Sat, Oct 1, 2016 at 3:22 PM, Anton Khirnov wrote: > The filtergraph's existence is used in several places to mean that the > filtergraph is fully configured. This causes problems if it's allocated, > but the initialization fails (e.g. if a non-existent filter is >

<    2   3   4   5   6   7   8   9   10   11   >