Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: refine EXT-X-BYTERANGE support for segments

2016-09-16 Thread Steven Liu
2016-09-15 20:31 GMT+08:00 Steven Liu : > > > 2016-09-14 6:51 GMT+08:00 Steven Liu : > >> >> >> 2016-09-14 2:53 GMT+08:00 Moritz Barsnick : >> >>> On Tue, Sep 13, 2016 at 11:25:39 +0800, Steven Liu wrote: >>> > +int byterange_mode = hls->flags & (HLS_SINGLE_FILE) || >>> hls->max_seg_size > 0;

Re: [FFmpeg-devel] [PATCH]lavc/libvpxenc: Avaoid vp8 transparency encoding with auto-alt-ref

2016-09-16 Thread James Zern
On Fri, Sep 16, 2016 at 4:50 PM, Carl Eugen Hoyos wrote: > 2016-09-17 1:48 GMT+02:00 James Zern: >> On Fri, Sep 16, 2016 at 5:23 AM, Carl Eugen Hoyos wrote: > >>> Attached patch fixes ticket #5815. >> >>> +if (ctx->auto_alt_ref && ctx->is_alpha && avctx->codec_id == >>> AV_CODEC_ID_VP8) { >>>

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Lou Logan
On Thu, 15 Sep 2016 14:36:32 -0300, James Almer wrote: > There have been some complains from developers about keeping both > SDL1 and SDL2 support at the same time even after all SDL1 code > has been already ported, and especially since it makes maintaining > ffplay more complex, so seeing the wat

Re: [FFmpeg-devel] [PATCH] doc/muxers: add flv muxer document into doc/muxers

2016-09-16 Thread Lou Logan
On Fri, 16 Sep 2016 09:50:25 +0800, Steven Liu wrote: > From e94de46102a57be1988cc451e1b610b404453c81 Mon Sep 17 00:00:00 2001 > From: Steven Liu > Date: Fri, 16 Sep 2016 08:58:33 +0800 > Subject: [PATCH] doc/muxers: add flv muxer document into doc/muxers > > add flvflags aac_seq_header_detect a

Re: [FFmpeg-devel] [PATCH]lavc/libvpxenc: Avaoid vp8 transparency encoding with auto-alt-ref

2016-09-16 Thread Carl Eugen Hoyos
2016-09-17 1:48 GMT+02:00 James Zern: > On Fri, Sep 16, 2016 at 5:23 AM, Carl Eugen Hoyos wrote: >> Attached patch fixes ticket #5815. > >> +if (ctx->auto_alt_ref && ctx->is_alpha && avctx->codec_id == >> AV_CODEC_ID_VP8) { >> +av_log(avctx, AV_LOG_ERROR, "Transparency encoding with

Re: [FFmpeg-devel] [PATCH]lavc/libvpxenc: Avaoid vp8 transparency encoding with auto-alt-ref

2016-09-16 Thread James Zern
On Fri, Sep 16, 2016 at 5:23 AM, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5815. > > +if (ctx->auto_alt_ref && ctx->is_alpha && avctx->codec_id == > AV_CODEC_ID_VP8) { > +av_log(avctx, AV_LOG_ERROR, "Transparency encoding with auto_alt_ref > does not work\n"); >

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 12:22 PM, Carl Eugen Hoyos wrote: > 2016-09-16 18:19 GMT+02:00 Ronald S. Bultje : > > Hi, > > > > On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos > > wrote: > > > >> 2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : > >> > >> > You want other developers to do more work

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Carl Eugen Hoyos
2016-09-16 18:24 GMT+02:00 Josh de Kock : > On 16/09/2016 17:22, Carl Eugen Hoyos wrote: >> >> 2016-09-16 18:19 GMT+02:00 Ronald S. Bultje : >>> >>> On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos >>> wrote: >>> 2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : > You want other develop

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Josh de Kock
On 16/09/2016 17:22, Carl Eugen Hoyos wrote: 2016-09-16 18:19 GMT+02:00 Ronald S. Bultje : Hi, On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos wrote: 2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : You want other developers to do more work Of which work are you talking about? Installin

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Carl Eugen Hoyos
2016-09-16 18:19 GMT+02:00 Ronald S. Bultje : > Hi, > > On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos > wrote: > >> 2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : >> >> > You want other developers to do more work >> >> Of which work are you talking about? > > Installing libsdl1. I don't unders

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 12:16 PM, Carl Eugen Hoyos wrote: > 2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : > > > You want other developers to do more work > > Of which work are you talking about? Installing libsdl1. Ronald ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Carl Eugen Hoyos
2016-09-16 16:07 GMT+02:00 Hendrik Leppkes : > You want other developers to do more work Of which work are you talking about? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Hendrik Leppkes
On Fri, Sep 16, 2016 at 2:48 PM, Carl Eugen Hoyos wrote: > 2016-09-16 14:21 GMT+02:00 Rostislav Pehlivanov : > >> I'd rather have SDL1 support dropped. ffplay is the only thing on my >> machine which uses libsdl1.2 > > And we already (everybody) agreed that we want to switch to SDL2 > so that you

Re: [FFmpeg-devel] [PATCHv2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-16 Thread Clément Bœsch
On Fri, Sep 16, 2016 at 03:20:49PM +0200, Vittorio Giovara wrote: > This is the assumption that is made in pixel format conversion do > throughout the code (in particular swscale), and BT-specifications > mandate. > > Add a warning to inform the user that an automatic selection is being > made. >

Re: [FFmpeg-devel] [PATCHv2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 9:20 AM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > This is the assumption that is made in pixel format conversion do > throughout the code (in particular swscale), and BT-specifications > mandate. > > Add a warning to inform the user that an automatic sel

[FFmpeg-devel] [PATCHv2] vf_colorspace: Interpret unspecified color range as limited range

2016-09-16 Thread Vittorio Giovara
This is the assumption that is made in pixel format conversion do throughout the code (in particular swscale), and BT-specifications mandate. Add a warning to inform the user that an automatic selection is being made. Signed-off-by: Vittorio Giovara --- libavfilter/vf_colorspace.c | 9 ++---

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 8:48 AM, Carl Eugen Hoyos wrote: > Do we allow developers to continue to use sdl1 to test on their > old (but supported) Linux distributions or do we forcibly stop > them from contributing here? All developers are welcome. You'll just have to install libsdl2 yoursel

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Carl Eugen Hoyos
2016-09-16 14:21 GMT+02:00 Rostislav Pehlivanov : > I'd rather have SDL1 support dropped. ffplay is the only thing on my > machine which uses libsdl1.2 And we already (everybody) agreed that we want to switch to SDL2 so that you (and others) do not have to use (or depend on) sdl1 anymore. But thi

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi folks, On Thu, Sep 15, 2016 at 1:36 PM, James Almer wrote: > The vote will end 1 week from now, simple majority wins, and > it's open only to those in the voting committee[1]. > Marton Balint, while not in the committee, should IMO also > have the chance to vote or at least comment since he's

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Ronald S. Bultje
Hi, On Fri, Sep 16, 2016 at 8:01 AM, Carl Eugen Hoyos wrote: > 2016-09-16 13:32 GMT+02:00 Paul B Mahol : > > >>> That's why i started this vote, to see how many devs are > >>> effectively in favor or against and make the final decision > >>> official. > >> > >> A vote on how evil the FFmpeg comm

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Rostislav Pehlivanov
On 15 September 2016 at 18:36, James Almer wrote: > There have been some complains from developers about keeping both > SDL1 and SDL2 support at the same time even after all SDL1 code > has been already ported, and especially since it makes maintaining > ffplay more complex, so seeing the waters

[FFmpeg-devel] [PATCH]lavc/libvpxenc: Avaoid vp8 transparency encoding with auto-alt-ref

2016-09-16 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #5815. Please comment, Carl Eugen From 3cf356b38a8ea6f9c98c36ad5cc7413dfa780e28 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Fri, 16 Sep 2016 14:21:09 +0200 Subject: [PATCH] lavc/libvpxenc: Avoid vp8 transparency encoding with auto-alt-ref. Fixes ticket

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Carl Eugen Hoyos
2016-09-16 13:32 GMT+02:00 Paul B Mahol : >>> That's why i started this vote, to see how many devs are >>> effectively in favor or against and make the final decision >>> official. >> >> A vote on how evil the FFmpeg community is wrt to >> new (and existing) contributors? Seriously? > > Proof? Wa

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Paul B Mahol
On 9/16/16, Carl Eugen Hoyos wrote: > 2016-09-15 19:55 GMT+02:00 James Almer : >> On 9/15/2016 2:51 PM, Carl Eugen Hoyos wrote: > >>> Didn't we all agree yesterday that this makes the transition >>> less painful? >> >> Not all. Several mentioned on IRC that they didn't like it. > > So development

Re: [FFmpeg-devel] [PATCH] avformat/http: remove duplicate user-agent option

2016-09-16 Thread Steven Liu
2016-09-16 19:07 GMT+08:00 Carl Eugen Hoyos : > 2016-09-16 4:43 GMT+02:00 Steven Liu : > > > +@item user-agent > > +This is a deprecated option, you can use user_agent instead it. > > Sorry for the nitpicking: Why do you move the first, unchanged line? > > Carl Eugen > > Because it will be remove,

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-16 Thread Carl Eugen Hoyos
2016-09-15 19:55 GMT+02:00 James Almer : > On 9/15/2016 2:51 PM, Carl Eugen Hoyos wrote: >> Didn't we all agree yesterday that this makes the transition >> less painful? > > Not all. Several mentioned on IRC that they didn't like it. So development discussion has moved to irc (again)? > That's w

Re: [FFmpeg-devel] [PATCH] avformat/http: remove duplicate user-agent option

2016-09-16 Thread Carl Eugen Hoyos
2016-09-16 4:43 GMT+02:00 Steven Liu : > +@item user-agent > +This is a deprecated option, you can use user_agent instead it. Sorry for the nitpicking: Why do you move the first, unchanged line? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffm

Re: [FFmpeg-devel] [PATCH] avformat/movenc: Make the packet check more tolerant

2016-09-16 Thread Michael Niedermayer
On Fri, Sep 16, 2016 at 11:45:54AM +0200, Paul B Mahol wrote: > On 9/16/16, Michael Niedermayer wrote: > > Depends-on "avformat/movenc: Check packet in mov_write_single_packet() too" > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/movenc.c | 31 --- > >

Re: [FFmpeg-devel] [PATCH 2/2] avformat/movenc: Check packet in mov_write_single_packet() too

2016-09-16 Thread Michael Niedermayer
On Fri, Sep 16, 2016 at 11:39:21AM +0200, Paul B Mahol wrote: > On 9/16/16, Michael Niedermayer wrote: > > Fixes assertion failure > > > > Found-by: durandal117 > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/movenc.c | 4 > > 1 file changed, 4 insertions(+) > > > > lgtm ap

Re: [FFmpeg-devel] [PATCH 1/2] avformat/movenc: Factor check_pkt() out

2016-09-16 Thread Michael Niedermayer
On Fri, Sep 16, 2016 at 11:38:57AM +0200, Paul B Mahol wrote: > On 9/16/16, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/movenc.c | 24 ++-- > > 1 file changed, 18 insertions(+), 6 deletions(-) > > > > lgtm applied thanks [...]

Re: [FFmpeg-devel] [PATCH] avformat/movenc: Make the packet check more tolerant

2016-09-16 Thread Paul B Mahol
On 9/16/16, Michael Niedermayer wrote: > Depends-on "avformat/movenc: Check packet in mov_write_single_packet() too" > Signed-off-by: Michael Niedermayer > --- > libavformat/movenc.c | 31 --- > 1 file changed, 16 insertions(+), 15 deletions(-) > lgtm ___

Re: [FFmpeg-devel] [PATCH] avutil/softfloat_ieee754: make all functions inline

2016-09-16 Thread Michael Niedermayer
On Thu, Sep 15, 2016 at 11:08:47PM -0300, James Almer wrote: > Removes "defined but not used" warnings > > Signed-off-by: James Almer > --- > libavutil/softfloat_ieee754.h | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) iam not the author/maintainer, but patch looks ok to

Re: [FFmpeg-devel] [PATCH 1/2] avformat/movenc: Factor check_pkt() out

2016-09-16 Thread Paul B Mahol
On 9/16/16, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/movenc.c | 24 ++-- > 1 file changed, 18 insertions(+), 6 deletions(-) > lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

Re: [FFmpeg-devel] [PATCH 2/2] avformat/movenc: Check packet in mov_write_single_packet() too

2016-09-16 Thread Paul B Mahol
On 9/16/16, Michael Niedermayer wrote: > Fixes assertion failure > > Found-by: durandal117 > Signed-off-by: Michael Niedermayer > --- > libavformat/movenc.c | 4 > 1 file changed, 4 insertions(+) > lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

2016-09-16 Thread Michael Niedermayer
On Thu, Sep 15, 2016 at 05:53:58PM -0800, Lou Logan wrote: > On Wed, 7 Sep 2016 14:27:07 +0200, Michael Niedermayer wrote: > > > yes one can ignore it but its alot more usefull if people keep in > > mind that theres patchwork. > > The patch submission checklist is lengthy. > > I'm fine with an a