Re: [FFmpeg-devel] [PATCH v4] avformat/movenc: Added an option to disable SIDX atom

2018-12-11 Thread Liu Steven
> 在 2018年12月12日,上午5:08,Ronak 写道: > >> >> On Dec 11, 2018, at 3:28 PM, Gyan wrote: >> >> >> On 12-12-2018 01:13 AM, Ronak wrote: >>> >>> Looks like I found out why: >>> https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/hlsenc.c#L790. The >>> hlsenc.c file overwrites whatever I

[FFmpeg-devel] [PATCH v2] lavc/qsvenc: set pict_type to be I for IDR frames.

2018-12-11 Thread Zhong Li
Signed-off-by: Zhong Li --- libavcodec/qsvenc.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index 931e994..96cf642 100644 --- a/libavcodec/qsvenc.c +++ b/libavcodec/qsvenc.c @@ -1377,11 +1377,10 @@ int

[FFmpeg-devel] [PATCH] avformat/hlsenc: remove duplicate operation at hls_write_header

2018-12-11 Thread Steven Liu
the options have set when avformat_init_output at hls_mux_init Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index bdd2a113bd..03a32b65d8 100644 ---

[FFmpeg-devel] [PATCH 1/2] avformat/dashenc : Refactored HLS media playlist related code

2018-12-11 Thread Karthick J
Made it as a separate function, so that it could be reused (in future) --- libavformat/dashenc.c | 135 +++--- 1 file changed, 75 insertions(+), 60 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 585b34cb97..f797b7bd1c 100644 ---

[FFmpeg-devel] [PATCH 2/2] avformat/dashenc: Added support for Low-latency HLS(LHLS)

2018-12-11 Thread Karthick J
Apple doesn't have an official spec for LHLS. Meanwhile hls.js player folks are trying to standardize a open LHLS spec. The draft spec is available in https://github.com/video-dev/hlsjs-rfcs/blob/lhls-spec/proposals/0001-lhls.md This option will also try to comply with the above open spec, till

Re: [FFmpeg-devel] [PATCH] lavc/qsvenc: set pict_type to be I for IDR frames.

2018-12-11 Thread Li, Zhong
> On Wed, Dec 12, 2018 at 2:13 PM Li, Zhong wrote: > > > > > > diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index > > > > aa7f347..8289a32 100644 > > > > --- a/libavcodec/qsvenc.c > > > > +++ b/libavcodec/qsvenc.c > > > > @@ -1378,10 +1378,11 @@ int ff_qsv_encode(AVCodecContext >

[FFmpeg-devel] [PATCH] lavc/qsv_hevc: correct QSV HEVC default plugin on Windows

2018-12-11 Thread Zhong Li
1. Old logic meaned: everywhere, except Windows, ffmpeg has to use HW acceleration, but on Windows ffmpeg has to use (unavailable) software HEVC by default 2. Software HEVC is available only if you provide corresponding software MediaSDK library, which isn't provided with ffmpeg. More information

Re: [FFmpeg-devel] [PATCH] lavc/qsvenc: set pict_type to be I for IDR frames.

2018-12-11 Thread myp...@gmail.com
On Wed, Dec 12, 2018 at 2:13 PM Li, Zhong wrote: > > > > diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index > > > aa7f347..8289a32 100644 > > > --- a/libavcodec/qsvenc.c > > > +++ b/libavcodec/qsvenc.c > > > @@ -1378,10 +1378,11 @@ int ff_qsv_encode(AVCodecContext *avctx, > > >

Re: [FFmpeg-devel] [PATCH] lavc/qsvenc: set pict_type to be I for IDR frames.

2018-12-11 Thread Li, Zhong
> > diff --git a/libavcodec/qsvenc.c b/libavcodec/qsvenc.c index > > aa7f347..8289a32 100644 > > --- a/libavcodec/qsvenc.c > > +++ b/libavcodec/qsvenc.c > > @@ -1378,10 +1378,11 @@ int ff_qsv_encode(AVCodecContext *avctx, > > QSVEncContext *q, > > new_pkt.size = bs->DataLength; > > > >

Re: [FFmpeg-devel] [PATCH] ffmpeg: implement input file stream disabling

2018-12-11 Thread Gyan
On 12-12-2018 03:40 AM, Michael Niedermayer wrote: On Mon, Dec 10, 2018 at 08:57:44PM +0530, Gyan wrote: At Michael's suggestion, this patch lets -vn/-an/-sn/-dn work for input files. Individual streams can still be let through e.g.     ffmpeg -an -discard:a:1 none -i file ... will let

Re: [FFmpeg-devel] [PATCH] swscale/ppc: Move VSX-using code to its own file

2018-12-11 Thread Michael Niedermayer
On Tue, Dec 11, 2018 at 09:10:02AM +0200, Lauri Kasanen wrote: > On Thu, 6 Dec 2018 21:47:18 +0100 > Michael Niedermayer wrote: > > > On Tue, Dec 04, 2018 at 02:27:22PM +0100, Michael Niedermayer wrote: > > > > > > On Mon, Dec 03, 2018 at 09:24:47AM +0200, Lauri Kasanen wrote: > > > > > > > Also

Re: [FFmpeg-devel] [PATCH 3/3] Add ADPCM IMA CRYO APC encoder

2018-12-11 Thread Michael Niedermayer
On Mon, Dec 10, 2018 at 12:32:51PM +0100, Tomas Härdin wrote: > > Changelog |1 + > libavcodec/adpcmenc.c | 33 + > libavcodec/allcodecs.c |1 + > libavcodec/version.h |4 ++-- > tests/fate/acodec.mak

Re: [FFmpeg-devel] [PATCH] ffmpeg: implement input file stream disabling

2018-12-11 Thread Michael Niedermayer
On Mon, Dec 10, 2018 at 08:57:44PM +0530, Gyan wrote: > At Michael's suggestion, this patch lets -vn/-an/-sn/-dn work for input > files. Individual streams can still be let through e.g. > >     ffmpeg -an -discard:a:1 none -i file ... > > will let (only) the 2nd audio stream be seen during

Re: [FFmpeg-devel] [PATCH v4] avformat/movenc: Added an option to disable SIDX atom

2018-12-11 Thread Ronak
> On Dec 11, 2018, at 3:28 PM, Gyan wrote: > > > On 12-12-2018 01:13 AM, Ronak wrote: >> >> Looks like I found out why: >> https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/hlsenc.c#L790. The >> hlsenc.c file overwrites whatever I pass in the command line... > > That is a bug but

[FFmpeg-devel] [PATCH 2/2] avcodec/gif: add support for alpha

2018-12-11 Thread Paul B Mahol
Fixes #6813. Signed-off-by: Paul B Mahol --- libavcodec/gif.c | 168 --- 1 file changed, 144 insertions(+), 24 deletions(-) diff --git a/libavcodec/gif.c b/libavcodec/gif.c index ef04fabe86..798953b8c6 100644 --- a/libavcodec/gif.c +++

Re: [FFmpeg-devel] [PATCH v4] avformat/movenc: Added an option to disable SIDX atom

2018-12-11 Thread Gyan
On 12-12-2018 01:13 AM, Ronak wrote: Looks like I found out why: https://github.com/FFmpeg/FFmpeg/blob/master/libavformat/hlsenc.c#L790. The hlsenc.c file overwrites whatever I pass in the command line... That is a bug but there's still an error in your command. It should be,    

[FFmpeg-devel] Add alternative delogo algorithm for my donation

2018-12-11 Thread uwe . freese
Hello, I would like that the delogo filter is improved by an alternative way of removing the logo. It's the "UGLARM" mode known from Virtual Dub's DeLogo filter and ffdshow. I used it many years (under Windows), but now moving to Linux miss it using ffmpeg as encoding tool. Since an ffmpeg

Re: [FFmpeg-devel] [PATCH v4] avformat/movenc: Added an option to disable SIDX atom

2018-12-11 Thread Ronak
> On Dec 11, 2018, at 1:42 PM, Ronak wrote: > > > >> On Dec 11, 2018, at 12:35 AM, Jeyapal, Karthick wrote: >> >> >> >> On 12/11/18, 9:07 AM, "Ronak" wrote: >> >>> On Dec 6, 2018, at 4:47 AM, Tobias Rapp wrote: On 06.12.2018 08:28, Karthick J wrote: > --- >

Re: [FFmpeg-devel] [PATCH v4] avformat/movenc: Added an option to disable SIDX atom

2018-12-11 Thread Ronak
> On Dec 11, 2018, at 12:35 AM, Jeyapal, Karthick wrote: > > > > On 12/11/18, 9:07 AM, "Ronak" wrote: > >> >>> On Dec 6, 2018, at 4:47 AM, Tobias Rapp wrote: >>> >>> On 06.12.2018 08:28, Karthick J wrote: --- [...] >>> >>> Looks OK now, no more comments from my side. >> >>

Re: [FFmpeg-devel] [PATCH] fix for transparencies in animated gifs

2018-12-11 Thread Paul B Mahol
Hi, On 11/27/17, Carl Eugen Hoyos wrote: > 2017-11-27 18:50 GMT+01:00 Bjorn Roche : >> Sorry for the very delayed response, Carl: >> >> On Tue, Nov 21, 2017 at 6:55 PM, Carl Eugen Hoyos >> wrote: >> >>> 2017-11-21 18:53 GMT+01:00 Bjorn Roche : >>> > Support for transparencies in animated gifs

Re: [FFmpeg-devel] [PATCH]tests/api-flac-test: Rename NUMBER_OF_FRAMES as NUMBER_OF_AUDIO_FRAMES

2018-12-11 Thread Carl Eugen Hoyos
2018-12-04 2:57 GMT+01:00, Michael Niedermayer : > On Mon, Dec 03, 2018 at 01:17:42AM +0100, Carl Eugen Hoyos wrote: >> Hi! >> >> Both aix and sunos define NUMBER_OF_FRAMES in a system header (as >> _NUMBER_OF_FRAMES), attached patch fixes a warning when running fate. >> >> Please comment, Carl

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Carl Eugen Hoyos
2018-12-11 17:40 GMT+01:00, Paul B Mahol : > On 12/11/18, Carl Eugen Hoyos wrote: >> 2018-12-11 17:37 GMT+01:00, Paul B Mahol : >>> On 12/11/18, Paul B Mahol wrote: On 12/11/18, Carl Eugen Hoyos wrote: > 2018-12-11 17:18 GMT+01:00, Paul B Mahol : >> On 12/11/18, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Paul B Mahol
On 12/11/18, Carl Eugen Hoyos wrote: > 2018-12-11 17:37 GMT+01:00, Paul B Mahol : >> On 12/11/18, Paul B Mahol wrote: >>> On 12/11/18, Carl Eugen Hoyos wrote: 2018-12-11 17:18 GMT+01:00, Paul B Mahol : > On 12/11/18, Carl Eugen Hoyos wrote: >> 2018-12-11 16:02 GMT+01:00, Paul B

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Carl Eugen Hoyos
2018-12-11 17:37 GMT+01:00, Paul B Mahol : > On 12/11/18, Paul B Mahol wrote: >> On 12/11/18, Carl Eugen Hoyos wrote: >>> 2018-12-11 17:18 GMT+01:00, Paul B Mahol : On 12/11/18, Carl Eugen Hoyos wrote: > 2018-12-11 16:02 GMT+01:00, Paul B Mahol : >> Now "-c copy" works. > >

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Paul B Mahol
On 12/11/18, Paul B Mahol wrote: > On 12/11/18, Carl Eugen Hoyos wrote: >> 2018-12-11 17:18 GMT+01:00, Paul B Mahol : >>> On 12/11/18, Carl Eugen Hoyos wrote: 2018-12-11 16:02 GMT+01:00, Paul B Mahol : > Now "-c copy" works. Please mention ticket #6640. When using

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Paul B Mahol
On 12/11/18, Carl Eugen Hoyos wrote: > 2018-12-11 17:18 GMT+01:00, Paul B Mahol : >> On 12/11/18, Carl Eugen Hoyos wrote: >>> 2018-12-11 16:02 GMT+01:00, Paul B Mahol : Now "-c copy" works. >>> >>> Please mention ticket #6640. >>> >>> When using the commands there, the output file has a

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Carl Eugen Hoyos
2018-12-11 17:18 GMT+01:00, Paul B Mahol : > On 12/11/18, Carl Eugen Hoyos wrote: >> 2018-12-11 16:02 GMT+01:00, Paul B Mahol : >>> Now "-c copy" works. >> >> Please mention ticket #6640. >> >> When using the commands there, the output file has a different >> byte at position 0xbb06 (0x0a ->

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Paul B Mahol
On 12/11/18, Carl Eugen Hoyos wrote: > 2018-12-11 16:02 GMT+01:00, Paul B Mahol : >> Now "-c copy" works. > > Please mention ticket #6640. > > When using the commands there, the output file has a different > byte at position 0xbb06 (0x0a -> 0x05), do you know why? Dunno, md5 of decoded output is

Re: [FFmpeg-devel] [PATCH] fate: add prog suffix in fate-mov-mp4-with-mov-in24-ver

2018-12-11 Thread Gyan
On 11-12-2018 02:18 PM, Gyan wrote: On 10-12-2018 08:53 PM, Gyan wrote: Maybe we should set a ffprobe variable in the fate Makefile, like done for ffmpeg on ln 8. Gyan Will push tonight if no objections. Pushed as 3b825b2f81e74bf9709d376f61fd1bf9a394f6ad Gyan

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Carl Eugen Hoyos
2018-12-11 16:02 GMT+01:00, Paul B Mahol : > Now "-c copy" works. Please mention ticket #6640. When using the commands there, the output file has a different byte at position 0xbb06 (0x0a -> 0x05), do you know why? Carl Eugen ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] avcodec: rewrite gif muxing and encoding

2018-12-11 Thread Carl Eugen Hoyos
2018-12-11 15:12 GMT+01:00, Paul B Mahol : > Now "-c copy" works. Please mention ticket #6640. The last frame seems missing, at least with the commands from this ticket. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] avcodec/h264_refs: reset MMCO when invalid mmco code is found

2018-12-11 Thread Paul B Mahol
On 12/11/18, Carl Eugen Hoyos wrote: > 2018-12-10 23:41 GMT+01:00, Mark Thompson : >> On 09/12/2018 14:21, Mark Thompson wrote: >>> On 09/12/2018 13:54, Paul B Mahol wrote: On 12/9/18, Mark Thompson wrote: > On 09/12/2018 08:52, Paul B Mahol wrote: >> On 12/7/18, Paul B Mahol

Re: [FFmpeg-devel] Separated FFserver code

2018-12-11 Thread Carl Eugen Hoyos
2018-11-30 15:48 GMT+01:00, Harshil Makwana : > Hello All, > > I have created separate ffserver code which can be used for rtsp server only > with no other dependency, > > Have a look into it: > > Https://github.com/harshil1991/ffserver.git Why did you remove the copyright notice? Carl Eugen

Re: [FFmpeg-devel] [RFC]lavc/mjpegenc_common: Fix aspect ratio

2018-12-11 Thread Kevin Wheatley
I guess it depends if you think that it is better to align with defacto behaviour (and maybe clarify/correct the specifications) or follow the specs and have users grumble about not matching the behaviour of 'applications X, Y and Z', I'm pretty certain Photoshop won't easily change its behaviour.

Re: [FFmpeg-devel] [PATCH] fate: add prog suffix in fate-mov-mp4-with-mov-in24-ver

2018-12-11 Thread Gyan
On 10-12-2018 08:53 PM, Gyan wrote: Maybe we should set a ffprobe variable in the fate Makefile, like done for ffmpeg on ln 8. Gyan Will push tonight if no objections. Gyan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org