Re: [libav-devel] [PATCH] [PATCH] hevc : Play HEVC TS streams which have stream_type 0x27.

2014-02-17 Thread Luca Barbato
On 18/02/14 03:37, Logan Y. Jeon wrote: > From: "Logan Y. Jeon" The code doesn't seem bad, but should be split in 3 like this: > --- > libavcodec/libx265.c|3 ++- x265: set internal bit depth > libavformat/mpeg.c |4 > libavformat/mpeg.h |1 + mpeg: Support HEV

Re: [libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Martin Storsjö
On Mon, 17 Feb 2014, Diego Biurrun wrote: --- Now setting explicit dependencies for w32threads. configure | 1 + libavutil/atomic.c | 6 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 292da47..841c39a 100755 --- a/configure +++ b/config

Re: [libav-devel] [PATCH] [PATCH] hevc : Play HEVC TS streams which have stream_type 0x27.

2014-02-17 Thread Hendrik Leppkes
On Tue, Feb 18, 2014 at 3:49 AM, Tim Walker wrote: > On 18 Feb 2014, at 03:37, Logan Y. Jeon wrote: > >> @@ -102,7 +102,8 @@ static av_cold int libx265_encode_init(AVCodecContext >> *avctx) >> ctx->params->frameRate = (int) (avctx->time_base.den / >> avctx->time_base.num); >> ctx-

Re: [libav-devel] [PATCH 2/2] ac3enc: allow Dolby Pro Logic IIz as the Dolby Surround EX mode.

2014-02-17 Thread Justin Ruggles
On 02/17/2014 09:39 PM, Tim Walker wrote: > This is actually defined in the A/52 specification. > --- > libavcodec/ac3enc.h | 1 + > libavcodec/ac3enc_opts_template.c | 3 ++- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/ac3enc.h b/libavcodec/ac3enc.

Re: [libav-devel] [PATCH 1/2] ac3enc: allow Dolby Pro Logic II as a preferred downmix mode.

2014-02-17 Thread Timothy Gu
On Mon, Feb 17, 2014 at 6:39 PM, Tim Walker wrote: > Some encoders already use this value even > though it's reserved in the A/52 specification. > --- > libavcodec/ac3enc.h | 1 + > libavcodec/ac3enc_opts_template.c | 3 ++- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff

Re: [libav-devel] [PATCH] [PATCH] hevc : Play HEVC TS streams which have stream_type 0x27.

2014-02-17 Thread Tim Walker
On 18 Feb 2014, at 03:37, Logan Y. Jeon wrote: > @@ -102,7 +102,8 @@ static av_cold int libx265_encode_init(AVCodecContext > *avctx) > ctx->params->frameRate = (int) (avctx->time_base.den / > avctx->time_base.num); > ctx->params->sourceWidth = avctx->width; > ctx->params->

[libav-devel] [PATCH 1/2] ac3enc: allow Dolby Pro Logic II as a preferred downmix mode.

2014-02-17 Thread Tim Walker
Some encoders already use this value even though it's reserved in the A/52 specification. --- libavcodec/ac3enc.h | 1 + libavcodec/ac3enc_opts_template.c | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/ac3enc.h b/libavcodec/ac3enc.h index be9dcf2..a

[libav-devel] [PATCH 2/2] ac3enc: allow Dolby Pro Logic IIz as the Dolby Surround EX mode.

2014-02-17 Thread Tim Walker
This is actually defined in the A/52 specification. --- libavcodec/ac3enc.h | 1 + libavcodec/ac3enc_opts_template.c | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/ac3enc.h b/libavcodec/ac3enc.h index a66cd22..3c2b42d 100644 --- a/libavcodec/ac3enc.

[libav-devel] [PATCH] [PATCH] hevc : Play HEVC TS streams which have stream_type 0x27.

2014-02-17 Thread Logan Y. Jeon
From: "Logan Y. Jeon" --- libavcodec/libx265.c|3 ++- libavformat/mpeg.c |4 libavformat/mpeg.h |1 + libavformat/mpegts.c|1 + libavformat/mpegts.h|1 + libavformat/mpegtsenc.c |3 +++ 6 files changed, 12 insertions(+), 1 deletion(-) diff --gi

Re: [libav-devel] [PATCH 3/4] h264: deprecate very early unused profile

2014-02-17 Thread Tim Walker
On 17 Feb 2014, at 08:38, Hendrik Leppkes wrote: > On Mon, Feb 17, 2014 at 12:06 AM, Vittorio Giovara > wrote: >> On Sun, Feb 16, 2014 at 11:15 PM, Luca Barbato wrote: >>> On 16/02/14 23:08, Vittorio Giovara wrote: --- This profile got dropped in 2005, I think it should be not listed

Re: [libav-devel] [PATCH 4/5] configure: Move cpunop into ARCH_EXT_LIST_X86

2014-02-17 Thread Luca Barbato
On 02/11/13 17:21, Diego Biurrun wrote: > It is a processor feature, so it belongs there. > --- > configure |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mai

Re: [libav-devel] [PATCH 5/5] configure: Split x86 SIMD architecture extensions into separate list

2014-02-17 Thread Luca Barbato
On 02/11/13 17:21, Diego Biurrun wrote: > This avoids checking inline/external assembly dependencies for x86 > extensions that do not play a role in inline/external assembly. > --- > configure | 12 > 1 file changed, 8 insertions(+), 4 deletions(-) > Ok. _

Re: [libav-devel] [PATCH 3/5] configure: Split some logical blocks off from CONFIG_LIST and HAVE_LIST

2014-02-17 Thread Luca Barbato
On 02/11/13 17:21, Diego Biurrun wrote: > --- > configure | 184 > + > 1 file changed, 100 insertions(+), 84 deletions(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://li

Re: [libav-devel] [PATCH 2/5] configure: Move inet_aton check into network function check block

2014-02-17 Thread Luca Barbato
On 02/11/13 17:21, Diego Biurrun wrote: > --- > configure |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) probably ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 1/5] configure: miscellaneous cosmetics

2014-02-17 Thread Luca Barbato
On 02/11/13 17:21, Diego Biurrun wrote: > Sort lists of components. Sort function checks and group them in more > sensible blocks. Add some whitespace where it helps readability. > Consistently use double instead of single quotes around component lists. > --- > configure | 68 >

Re: [libav-devel] [PATCH 1/5] configure: miscellaneous cosmetics

2014-02-17 Thread Diego Biurrun
On Sat, Dec 21, 2013 at 04:24:35PM +0100, Diego Biurrun wrote: > On Fri, Dec 13, 2013 at 08:04:57PM +0100, Diego Biurrun wrote: > > On Sat, Nov 02, 2013 at 05:21:49PM +0100, Diego Biurrun wrote: > > > Sort lists of components. Sort function checks and group them in more > > > sensible blocks. Add s

Re: [libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Diego Biurrun
On Mon, Feb 17, 2014 at 05:53:32PM +0200, Martin Storsjö wrote: > On Mon, 17 Feb 2014, Diego Biurrun wrote: > >On Mon, Feb 17, 2014 at 03:31:20PM +0200, Martin Storsjö wrote: > >>On Mon, 17 Feb 2014, Diego Biurrun wrote: > > > >>Doing this kinda gives you a circular dependency, enabling 'threads' >

[libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Diego Biurrun
--- Now setting explicit dependencies for w32threads. configure | 1 + libavutil/atomic.c | 6 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 292da47..841c39a 100755 --- a/configure +++ b/configure @@ -1606,6 +1606,7 @@ atomics_gcc_if="

Re: [libav-devel] [PATCH 3/3] libavformat/mpegts: expose raw packet size

2014-02-17 Thread Leandro Dorileo
Hi Anton, Anton Khirnov wrote: From: Leandro Dorileo We cannot easily determine if an mpeg TS's packet size is DVHS, FEC or so on, for that we need to expose the internal raw_packet_size field. Signed-off-by: Anton Khirnov Tested with your changes, works just fine. Thank you. --- liba

Re: [libav-devel] [PATCH 2/3] AVOptions: add flags for read/read-only options

2014-02-17 Thread Leandro Dorileo
Anton Khirnov wrote: Tested-by: Leandro Dorileo --- doc/APIchanges |4 libavutil/opt.c | 14 +++--- libavutil/opt.h |9 + libavutil/version.h |4 ++-- 4 files changed, 26 insertions(+), 5 deletions(-) diff --git a/doc/APIchanges b/doc/AP

Re: [libav-devel] [PATCH 1/3] AVOptions: deprecate unused AV_OPT_FLAG_METADATA

2014-02-17 Thread Leandro Dorileo
Anton Khirnov wrote: It was never used since it was added and the things it was intended for are now exported differently. Just a side note, you'll need to redo the version bumps since Tim has bumped it last week. Tested-by: Leandro Dorileo --- doc/APIchanges |3 +++ libavutil

[libav-devel] [PATCH] h264: document Picture pointers

2014-02-17 Thread Vittorio Giovara
--- After a chat with Janne and Anton (thanks for the explanation). Please correct me if I misunderstood Vittorio libavcodec/h264.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/h264.h b/libavcodec/h264.h index 69a46ca..bd05599 100644 --- a/libavcodec/h264.h

Re: [libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Martin Storsjö
On Mon, 17 Feb 2014, Diego Biurrun wrote: On Mon, Feb 17, 2014 at 03:31:20PM +0200, Martin Storsjö wrote: On Mon, 17 Feb 2014, Diego Biurrun wrote: Doing this kinda gives you a circular dependency, enabling 'threads' requires one of the elements in $THREADS_LIST to be enabled. But if 'thread

[libav-devel] [PATCH] h264: Informative error reporting

2014-02-17 Thread Luca Barbato
--- libavcodec/h264.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index 421e12a..9cc32e9 100644 --- a/libavcodec/h264.c +++ b/libavcodec/h264.c @@ -3368,7 +3368,7 @@ static int decode_slice_header(H264Context *h, H264Context

Re: [libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Diego Biurrun
On Mon, Feb 17, 2014 at 03:31:20PM +0200, Martin Storsjö wrote: > On Mon, 17 Feb 2014, Diego Biurrun wrote: > >--- a/configure > >+++ b/configure > >@@ -1606,7 +1606,7 @@ atomics_gcc_if="sync_val_compare_and_swap" > >atomics_suncc_if="atomic_cas_ptr machine_rw_barrier" > >atomics_win32_if="MemoryBa

Re: [libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Martin Storsjö
On Mon, 17 Feb 2014, Diego Biurrun wrote: --- Now without a silly leftover line in configure. configure | 2 +- libavutil/atomic.c | 6 +- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 292da47..cb566c0 100755 --- a/configure +++ b/configu

[libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Diego Biurrun
--- Now without a silly leftover line in configure. configure | 2 +- libavutil/atomic.c | 6 +- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 292da47..cb566c0 100755 --- a/configure +++ b/configure @@ -1606,7 +1606,7 @@ atomics_gcc_if="s

Re: [libav-devel] [PATCH] atomics: Check dependencies for enabling threading at the configure stage

2014-02-17 Thread Diego Biurrun
On Fri, Dec 27, 2013 at 11:41:36AM +0200, Martin Storsjö wrote: > On Fri, 27 Dec 2013, Diego Biurrun wrote: > >On Thu, Dec 26, 2013 at 07:42:05PM +0200, Martin Storsjö wrote: > >>On Wed, 25 Dec 2013, Diego Biurrun wrote: > >>>--- > >>> > >>>Disregard the previous version, which was completely broke