Re: [libav-devel] x264: Allow yuvj422p and yuvj444p

2015-03-18 Thread Bradley Sepos
On Tue Mar 17 14:24:54 CET 2015, Vittorio Giovara gmail.com> wrote: looks good, although I have to adjust authorship and commit log a little libx264: Allow full-range yuv422 and yuv444 pixel formats thanks for the contribution No worries, everything looks appropriate to me. Glad it's useful. B

Re: [libav-devel] [PATCH 1/8] movenc: Heuristically set the duration of the last sample in a fragment if not set

2015-03-18 Thread Luca Barbato
On 18/03/15 19:16, Martin Storsjö wrote: On Mon, 16 Mar 2015, Luca Barbato wrote: On 16/03/15 14:44, Martin Storsjö wrote: Even if this is a guess, it is way better than writing a zero duration of the last sample in a fragment (because if the duration is zero, the first sample of the next frag

Re: [libav-devel] [PATCH 4/8] movenc: Write pts timestamps in tfxd, instead of dts timestamps

2015-03-18 Thread Luca Barbato
On 16/03/15 14:44, Martin Storsjö wrote: This matches what we write in tfra and tfrf since 9cbf70fa0e. --- libavformat/movenc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index b9e75f4..bc5370e 100644 --- a/libavformat/move

Re: [libav-devel] [PATCH 5/8] movenc: Only adjust the cts offset at the start of fragments if necessary

2015-03-18 Thread Luca Barbato
On 16/03/15 14:44, Martin Storsjö wrote: Adjusting it is only necessary when a sidx/tfrf/tfxd atom already has been written for the previous fragment (since the sidx/tfrf/tfxd atoms include the duration between the first pts of the previous fragment, to the first pts of the new fragment). --- l

Re: [libav-devel] [PATCH 6/8] movenc: Move sidx edit list timestamp adjustment into a block

2015-03-18 Thread Luca Barbato
On 16/03/15 14:44, Martin Storsjö wrote: When reading these values from track->frag_info, the same adjustment has already been done. --- libavformat/movenc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) Ok. ___ libav-devel mailing

Re: [libav-devel] [PATCH 7/8] movenc: Don't assume that fragment durations in pts is equal to duration in dts

2015-03-18 Thread Luca Barbato
On 16/03/15 14:44, Martin Storsjö wrote: For strict CFR, they should be pretty much equal, but if the stream is VFR, there can be a sometimes significant difference. Calculate the pts duration separately, used in sidx atoms and for tfrf/tfxd boxes in smooth streaming ismv files. Probably Ok.

Re: [libav-devel] [PATCH 8/8] ismindex: Calculate the pts duration of trun atoms, not the dts duration

2015-03-18 Thread Luca Barbato
On 16/03/15 14:44, Martin Storsjö wrote: Since the duration is compared to the tfra durations/intervals which are expressed in pts, calculate that here as well. --- tools/ismindex.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) Looks fine to me.

Re: [libav-devel] [PATCH 1/8] movenc: Heuristically set the duration of the last sample in a fragment if not set

2015-03-18 Thread Martin Storsjö
On Mon, 16 Mar 2015, Luca Barbato wrote: On 16/03/15 14:44, Martin Storsjö wrote: Even if this is a guess, it is way better than writing a zero duration of the last sample in a fragment (because if the duration is zero, the first sample of the next fragment will have the same timestamp as the l

[libav-devel] [PATCH] cmdutils: Add a stream specifier to map usable streams

2015-03-18 Thread Luca Barbato
It drops everything that cannot be used for re-encoding and/or streamcopy. --- cmdutils.c | 19 +++ doc/avtools-common-opts.texi | 3 +++ 2 files changed, 22 insertions(+) diff --git a/cmdutils.c b/cmdutils.c index e01ad24..2b0a833 100644 --- a/cmdutils.c +++ b/

Re: [libav-devel] [PATCH 2/2] pixfmt: Update documentation with prefixed pixel formats

2015-03-18 Thread Luca Barbato
On 18/03/15 12:10, Vittorio Giovara wrote: --- libavutil/pixfmt.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 1/2] pixdesc: Replace a few leftover instances of non-AV prefixed flags

2015-03-18 Thread Luca Barbato
On 18/03/15 12:10, Vittorio Giovara wrote: --- libavutil/pixdesc.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index bd72c04..f991370 100644 --- a/libavutil/pixdesc.c +++ b/libavutil/pixdesc.c @@ -1462,7 +1462,8 @@ cons

[libav-devel] [PATCH 1/2] pixdesc: Replace a few leftover instances of non-AV prefixed flags

2015-03-18 Thread Vittorio Giovara
--- libavutil/pixdesc.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index bd72c04..f991370 100644 --- a/libavutil/pixdesc.c +++ b/libavutil/pixdesc.c @@ -1462,7 +1462,8 @@ const AVPixFmtDescriptor av_pix_fmt_descriptors[AV

[libav-devel] [PATCH 2/2] pixfmt: Update documentation with prefixed pixel formats

2015-03-18 Thread Vittorio Giovara
--- libavutil/pixfmt.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavutil/pixfmt.h b/libavutil/pixfmt.h index 4763583..d1bf367 100644 --- a/libavutil/pixfmt.h +++ b/libavutil/pixfmt.h @@ -34,17 +34,17 @@ * Pixel format. * * @note - * PIX_FMT_RGB32