[FFmpeg-devel] [PATCH 2/8] lavf/mux: run AVCodec::deinit if write_header fails

--- libavformat/mux.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index 3d8ea45..b5131f1 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -477,8 +477,11 @@ int avformat_write_header(AVFormatContext *s, AVDictionary

[FFmpeg-devel] [PATCH 3/8] lavf/mux: add avformat_init_output

This allows a consumer to run the muxer's init function without actually writing the header, which is useful in chained muxers that support automatic bitstream filtering. --- libavformat/avformat.h | 30 +++-- libavformat/internal.h | 10 + libavformat/mux.c | 59

[FFmpeg-devel] [PATCH 5/8] lavf/segment: fix writing separate header with auto BSF

--- libavformat/segment.c | 55 ++- 1 file changed, 41 insertions(+), 14 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 1fffb7b..1578bdb 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -260,6 +260,7

[FFmpeg-devel] [PATCH 4/8] lavf/segment: add deinit function

--- libavformat/segment.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index df6f4b5..1fffb7b 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -627,8 +627,9 @@ static int

[FFmpeg-devel] [PATCH 7/8] lavf/dashenc: add deinit function

--- libavformat/dashenc.c | 51 +-- 1 file changed, 17 insertions(+), 34 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 519f9c4..0848052 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -580,16

[FFmpeg-devel] [PATCH 6/8] lavf/movenc: add deinit function

--- libavformat/movenc.c | 63 ++-- 1 file changed, 27 insertions(+), 36 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 5acb9af..1e1dc84 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5263,21 +5263,18

[FFmpeg-devel] [PATCH 8/8] lavf/movenc+dashenc: add automatic bitstream filtering

This is disabled by default when the empty_moov flag is enabled --- libavformat/dashenc.c | 41 +++- libavformat/movenc.c | 103 +++--- 2 files changed, 119 insertions(+), 25 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH 1/8] lavf: add a flag to enable/disable automatic bitstream filtering

This is mostly useful for muxers that wrap other muxers, such as dashenc and segment. The actual duplicated bitstream filtering is largely harmless, but delaying the header can cause problems when the muxer intended the header to be written to a separate file. --- libavformat/avformat.h | 1

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

On Thu, 7 Apr 2016 11:44:20 -0400 nablet developer wrote: > Signed-off-by: nablet developer > --- > libavcodec/qsv.c | 64 > +-- > libavcodec/qsv.h | 53 +++ >

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

> On 13 Apr 2016, at 14:33, wm4 wrote: > > On Wed, 13 Apr 2016 14:14:37 +0600 > nablet developer wrote: > >>> Why would this API need to be exported? >> >> previously QuickSync was used only by libavcodec and its components - e.g. >> there are QSV

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

> On 11 Apr 2016, at 18:30, nablet developer wrote: > > >>> >>> diff --git a/libavcodec/qsv.c b/libavcodec/qsv.c >>> index 4c8e6b0..81d1f0c 100644 >>> --- a/libavcodec/qsv.c >>> +++ b/libavcodec/qsv.c >>> @@ -18,14 +18,19 @@ >>> * Foundation, Inc., 51 Franklin Street, Fifth

[FFmpeg-devel] [PATCH 1/2] lavc/mpegvideo_parser: export additional properties

--- libavcodec/mpegvideo_parser.c | 67 +++ 1 file changed, 67 insertions(+) diff --git a/libavcodec/mpegvideo_parser.c b/libavcodec/mpegvideo_parser.c index 1f74bfb..92ba24a 100644 --- a/libavcodec/mpegvideo_parser.c +++ b/libavcodec/mpegvideo_parser.c @@

[FFmpeg-devel] [PATCH 2/2] lavc/parser: export resolution and format in AVCodecContext if not already set

This allows avformat_find_stream_info to finish without opening a decoder in more cases. --- libavcodec/parser.c | 12 tests/ref/seek/vsynth_lena-mpeg1 | 38 +++--- 2 files changed, 31 insertions(+), 19 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

On 13/04/16 09:14, nablet developer wrote: > >> On 13 Apr 2016, at 14:08, wm4 wrote: >> >> On Thu, 7 Apr 2016 11:44:20 -0400 >> nablet developer > wrote: >> >>> Signed-off-by: nablet developer >>> --- >>>

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

> On 13 Apr 2016, at 14:48, Mark Thompson wrote: > > On 13/04/16 09:14, nablet developer wrote: >> >>> On 13 Apr 2016, at 14:08, wm4 wrote: >>> >>> On Thu, 7 Apr 2016 11:44:20 -0400 >>> nablet developer > wrote:

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

Le quintidi 25 germinal, an CCXXIV, wm4 a écrit : > Wouldn't it be better to keep it internal then? Sharing code between > the sub-libs is unfortunately messy Once again, I think we should merge the libs. (Without API change, of course; just the linking: -lffmpeg instead of -lavfilter -lavformat

Re: [FFmpeg-devel] [PATCH] Fix for H.264 configuration parsing

On Tue, Apr 12, 2016 at 11:47 PM, Carl Eugen Hoyos wrote: > > Could you provide such a stream? > > This is a stream I encoded using Android MediaCodec API and it breaks when I switch resolution. I have another stream that was encoded with a different encoder and there the

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

On Thu, Apr 7, 2016 at 5:44 PM, nablet developer wrote: > > +/** > + * Initialize a MSDK session > + * > + * Media SDK is based on sessions, so this is the prerequisite > + * initialization for HW acceleration. For Windows the session is > + * complete and ready to use, for

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

> On 13 Apr 2016, at 14:16, Hendrik Leppkes wrote: > > On Thu, Apr 7, 2016 at 5:44 PM, nablet developer wrote: >> >> +/** >> + * Initialize a MSDK session >> + * >> + * Media SDK is based on sessions, so this is the prerequisite >> + * initialization for

Re: [FFmpeg-devel] [PATCH]lavf/webpenc: Do not set alpha flag for yuv420p

On Tue, 12 Apr 2016 19:53:13 + (UTC) Carl Eugen Hoyos wrote: > wm4 googlemail.com> writes: > > > > -flags |= 2 + 16; > > > +flags |= 2; > > > +if (st->codecpar->format != AV_PIX_FMT_YUV420P) > > > +flags

[FFmpeg-devel] [PATCH v6 3/3] avformat/tee: Handling slave failure in tee muxer

From: Jan Sebechlebsky Adds per slave option 'onfail' to the tee muxer allowing an output to fail,so other slave outputs can continue. Signed-off-by: Jan Sebechlebsky --- Thanks for pointing on the problem with testing enum for negative

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

> On 13 Apr 2016, at 14:08, wm4 wrote: > > On Thu, 7 Apr 2016 11:44:20 -0400 > nablet developer > wrote: > >> Signed-off-by: nablet developer >> --- >> libavcodec/qsv.c | 64 >>

Re: [FFmpeg-devel] [PATCH] avcodec/qsv: export session management functionality

On Wed, 13 Apr 2016 14:14:37 +0600 nablet developer wrote: > > Why would this API need to be exported? > > previously QuickSync was used only by libavcodec and its components - e.g. > there are QSV encoder and decoders for AVC and MPEG-2. so it was OK that > QuickSync

Re: [FFmpeg-devel] [PATCH 1/2] avformat/framehash: don't share write_header code with framecrc muxers

On Wed, Apr 13, 2016 at 12:15:42AM -0300, James Almer wrote: > On 4/12/2016 11:31 PM, Michael Niedermayer wrote: > > On Tue, Apr 12, 2016 at 07:32:44PM -0300, James Almer wrote: [...] > > we never would want to use a old version for checking > > anything as it simply checks less at no advantage >

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/framehash: add extradata checksum

On Wed, Apr 13, 2016 at 01:54:25AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > I'm keeping the version code for the time being, unless people disagree. > Removing it if needed is trivial. > > libavformat/hashenc.c | 30 ++ > 1

Re: [FFmpeg-devel] [PATCH v2] avfilter/vf_drawtext: add optional default value to metadata function

On 08.04.2016 09:19, Tobias Rapp wrote: On 07.04.2016 20:45, Stefano Sabatini wrote: On date Thursday 2016-04-07 12:32:47 +0200, Tobias Rapp encoded: Signed-off-by: Tobias Rapp --- doc/filters.texi | 7 ++- libavfilter/vf_drawtext.c | 4 +++- 2 files

Re: [FFmpeg-devel] [PATCH v2] avfilter/vf_drawtext: add optional default value to metadata function

On 4/13/16, Tobias Rapp wrote: > On 08.04.2016 09:19, Tobias Rapp wrote: >> On 07.04.2016 20:45, Stefano Sabatini wrote: >>> On date Thursday 2016-04-07 12:32:47 +0200, Tobias Rapp encoded: Signed-off-by: Tobias Rapp ---

[FFmpeg-devel] Fate/exr : Add test for piz

Hello, In attach a patch to add fate test for piz compression (not cover now). Samples can be found here : https://we.tl/9WadnNpF4t (and need to be put inside fate-suite/exr) rgb_scanline_half_piz_bw : Replicate trouble of ticket #3932 rgb_scanline_half_piz_color : a more common piz file

Re: [FFmpeg-devel] [PATCH] avformat/framecrc: enable new output

On Wed, Apr 13, 2016 at 02:08:21AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > This change is not dependent on the framehash patches. I'm not going to > convert framecrc into a framehash alias for now. > The updated ref files should be the same as in

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/framehash: add extradata checksum

On Wed, Apr 13, 2016 at 12:28:23PM +0200, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 01:54:25AM -0300, James Almer wrote: > > Signed-off-by: James Almer > > --- > > I'm keeping the version code for the time being, unless people disagree. > > Removing it if needed is

Re: [FFmpeg-devel] [PATCH 1/2] lavc/mpegvideo_parser: export additional properties

On Wed, Apr 13, 2016 at 03:26:03AM -0500, Rodger Combs wrote: > --- > libavcodec/mpegvideo_parser.c | 67 > +++ > 1 file changed, 67 insertions(+) is this intended to change aspect of some files ? like

Re: [FFmpeg-devel] [PATCH 1/8] lavf: add a flag to enable/disable automatic bitstream filtering

On Wed, Apr 13, 2016 at 04:28:31AM -0500, Rodger Combs wrote: > This is mostly useful for muxers that wrap other muxers, such as dashenc > and segment. The actual duplicated bitstream filtering is largely harmless, > but delaying the header can cause problems when the muxer intended the header >

Re: [FFmpeg-devel] [PATCH 2/8] lavf/mux: run AVCodec::deinit if write_header fails

On Wed, Apr 13, 2016 at 04:28:32AM -0500, Rodger Combs wrote: > --- > libavformat/mux.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) should be ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who are best at talking, realize last or

[FFmpeg-devel] DVB Subtitles Live transcoding

Hiya All. We are having issues with DVB Subtitles. Our streams come from MUMUDVB and if played in VLC directly the subtitles appear as a subtitle stream that can be turned on and off. When we pass the stream through FFMpeg the subtitles no longer appear. We have tried using the map feature

Re: [FFmpeg-devel] [PATCH 4/8] lavf/segment: add deinit function

On Wed, Apr 13, 2016 at 04:28:34AM -0500, Rodger Combs wrote: > --- > libavformat/segment.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) still breaks libavformat/segment.c: In function ‘seg_init’: libavformat/segment.c:797:9: error: implicit declaration of function

Re: [FFmpeg-devel] DVB Subtitles Live transcoding

Liam Carter gmail.com> writes: > We are having issues with DVB Subtitles. Please read https://www.ffmpeg.org/contact.html (again). Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/8] lavf/mux: add avformat_init_output

On Wed, Apr 13, 2016 at 04:28:33AM -0500, Rodger Combs wrote: > This allows a consumer to run the muxer's init function without actually > writing the header, which is useful in chained muxers that support > automatic bitstream filtering. > --- > libavformat/avformat.h | 30

Re: [FFmpeg-devel] [PATCH 1/2] avformat/framehash: don't share write_header code with framecrc muxers

On 4/13/2016 6:56 AM, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 12:15:42AM -0300, James Almer wrote: >> On 4/12/2016 11:31 PM, Michael Niedermayer wrote: >>> On Tue, Apr 12, 2016 at 07:32:44PM -0300, James Almer wrote: > [...] >>> we never would want to use a old version for checking

Re: [FFmpeg-devel] Fate/exr : Add test for piz

On Wed, Apr 13, 2016 at 01:53:08PM +0200, Martin Vignali wrote: > Hello, > > In attach a patch to add fate test for piz compression (not cover now). > > Samples can be found here : https://we.tl/9WadnNpF4t > (and need to be put inside fate-suite/exr) > > rgb_scanline_half_piz_bw : Replicate

Re: [FFmpeg-devel] [PATCH] avcodec/wmalosslessdec: real 24bit support

Hi, 2016-04-12 22:53 GMT+02:00 Paul B Mahol : > -LLAudDSPContext dsp; ///< accelerated And later: > +static int scalarproduct_and_madd_int(int *v1, const int *v2, > + const int *v3, > +

Re: [FFmpeg-devel] DVB Subtitles Live transcoding

Hiya. Please accept my apologies i will post in user. Many Regards On 13 April 2016 at 17:01, Carl Eugen Hoyos wrote: > Liam Carter gmail.com> writes: > > > We are having issues with DVB Subtitles. > > Please read https://www.ffmpeg.org/contact.html (again). > > Carl Eugen

[FFmpeg-devel] [PATCH v3] avformat/framehash: add extradata checksum

Signed-off-by: James Almer --- libavformat/hashenc.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/libavformat/hashenc.c b/libavformat/hashenc.c index 614c815..4aa4fcf 100644 --- a/libavformat/hashenc.c +++ b/libavformat/hashenc.c @@ -149,6

[FFmpeg-devel] [PATCH] fate: add test for Ticket4816 (PNG decoding with alpha)

Signed-off-by: Michael Niedermayer --- tests/fate/video.mak|3 +++ tests/ref/fate/rgbapng-4816 |2 ++ 2 files changed, 5 insertions(+) create mode 100644 tests/ref/fate/rgbapng-4816 diff --git a/tests/fate/video.mak b/tests/fate/video.mak index

Re: [FFmpeg-devel] Fate/exr : Add test for piz

On Wed, Apr 13, 2016 at 06:48:33PM +0200, Martin Vignali wrote: > 2016-04-13 16:45 GMT+02:00 Michael Niedermayer : > > > On Wed, Apr 13, 2016 at 01:53:08PM +0200, Martin Vignali wrote: > > > Hello, > > > > > > In attach a patch to add fate test for piz compression (not

Re: [FFmpeg-devel] Fate/exr : Add test for piz

2016-04-13 16:45 GMT+02:00 Michael Niedermayer : > On Wed, Apr 13, 2016 at 01:53:08PM +0200, Martin Vignali wrote: > > Hello, > > > > In attach a patch to add fate test for piz compression (not cover now). > > > > Samples can be found here : https://we.tl/9WadnNpF4t > >

Re: [FFmpeg-devel] [PATCH v3] avformat/framehash: add extradata checksum

On Wed, Apr 13, 2016 at 11:01:21AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavformat/hashenc.c | 23 +++ > 1 file changed, 23 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] libavcodec/exr : Fix PXR24 uncompress

Hello, In attach a patch, to change the way PXR24 uncompress calc the expected size. My previous patch (fix PXR24 float) doesn't work when all channels of a file doesn't have the same pixel type. Now this patch calc the expected_len channel by channel. Comments welcome Martin Jokyo Images

Re: [FFmpeg-devel] [PATCH v3] avformat/framehash: add extradata checksum

On 4/13/2016 2:20 PM, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 11:01:21AM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavformat/hashenc.c | 23 +++ >> 1 file changed, 23 insertions(+) > > LGTM > > thx Pushed.

Re: [FFmpeg-devel] [Patch][GSoC] Motion Estimation filter

On Wed, Apr 13, 2016 at 07:24:42PM +, Davinder Singh wrote: > good vectors? how can i improve them? since it search for every possible > place, it should give best match. can you give more details, why > surrounding vectors need to be considered? the goal is to find the true motion that is

Re: [FFmpeg-devel] [PATCH 1/2] avformat/framehash: don't share write_header code with framecrc muxers

On Wed, Apr 13, 2016 at 04:49:50PM -0300, James Almer wrote: > On 4/12/2016 11:31 PM, Michael Niedermayer wrote: > > either way, IMO the latest version should be default (its compatible > > just has more fields) > > Except for the sidedata addition i sent last night, which adds one or more >

Re: [FFmpeg-devel] [Patch][GSoC] Motion Estimation filter

Am 13.04.16 um 22:01 schrieb Michael Niedermayer: > On Wed, Apr 13, 2016 at 07:24:42PM +, Davinder Singh wrote: >> good vectors? how can i improve them? since it search for every possible >> place, it should give best match. can you give more details, why >> surrounding vectors need to be

Re: [FFmpeg-devel] [Patch][GSoC] Motion Estimation filter

On Wed, Apr 13, 2016 at 10:01:35PM +0200, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 07:24:42PM +, Davinder Singh wrote: > > good vectors? how can i improve them? since it search for every possible > > place, it should give best match. can you give more details, why > > surrounding

Re: [FFmpeg-devel] [Patch][GSoC] Motion Estimation filter

good vectors? how can i improve them? since it search for every possible place, it should give best match. can you give more details, why surrounding vectors need to be considered? also i tried to compare it with export_mvs, they seems quite similar. the export_mvs ones, sometimes use multiple

Re: [FFmpeg-devel] [PATCH] avformat/framecrc: enable new output

On 4/13/2016 7:45 AM, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 02:08:21AM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> This change is not dependent on the framehash patches. I'm not going to >> convert framecrc into a framehash alias for now. >>

Re: [FFmpeg-devel] [PATCH] Implement hdcd filtering

On Tue, Apr 12, 2016 at 04:28:20PM +0200, Benjamin St wrote: > > > > Please compress the input sample with a lossless audio > > encoder. > > > Ok, input is know compressed with flac. flac file uploded [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The

Re: [FFmpeg-devel] [PATCH 2/2] avformat/framehash: add sidedata checksum

On 4/13/2016 8:44 PM, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 01:54:26AM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> Once these two patches go in I'll send one enabling version 2 output >> by default. >> >> libavformat/hashenc.c | 36

Re: [FFmpeg-devel] [PATCH] avformat/framehash: enable new output

On 4/13/2016 8:55 PM, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 02:22:16PM -0300, James Almer wrote: >> Also, since it's compatible with version 1, make it part of it instead of >> the new version 2. >> >> Signed-off-by: James Almer > > LGTM > thx Pushed.

Re: [FFmpeg-devel] [PATCH 1/2] lavc/mpegvideo_parser: export additional properties

This seems to be determined by whether information is copied from the codec context to codecpar for the last time before or after the input file's aspect ratio changes. I'm not sure what the "right" value to return here is, nor why this is different with this patch. > On Apr 13, 2016, at

Re: [FFmpeg-devel] [PATCH] Implement hdcd filtering

On Tue, Apr 12, 2016 at 04:28:20PM +0200, Benjamin St wrote: > > > > Please compress the input sample with a lossless audio > > encoder. > > > Ok, input is know compressed with flac. > > > Note that you also have to send a patch that adds the > > new license to allow fate to pass. > > Fate is

Re: [FFmpeg-devel] [PATCH v6 3/3] avformat/tee: Handling slave failure in tee muxer

On Wed, Apr 13, 2016 at 10:15:32AM +0300, sebechlebsky...@gmail.com wrote: > From: Jan Sebechlebsky > > Adds per slave option 'onfail' to the tee muxer allowing an output to > fail,so other slave outputs can continue. > > Signed-off-by: Jan Sebechlebsky

Re: [FFmpeg-devel] [Patch][GSoC] Motion Estimation filter

On Wed, Apr 13, 2016 at 10:30:35PM +0200, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 10:01:35PM +0200, Michael Niedermayer wrote: > > On Wed, Apr 13, 2016 at 07:24:42PM +, Davinder Singh wrote: > > > good vectors? how can i improve them? since it search for every possible > > >

Re: [FFmpeg-devel] [Patch][GSoC] Motion Estimation filter

On Wed, Apr 13, 2016 at 7:24 PM, Davinder Singh wrote: > good vectors? how can i improve them? since it search for every possible > place, it should give best match. can you give more details, why > surrounding vectors need to be considered? > > also i tried to compare it

Re: [FFmpeg-devel] [PATCH 1/2] avformat/framehash: don't share write_header code with framecrc muxers

On 4/12/2016 11:31 PM, Michael Niedermayer wrote: > either way, IMO the latest version should be default (its compatible > just has more fields) Except for the sidedata addition i sent last night, which adds one or more hashes to a given frame's line, the new output is indeed compatible to what

Re: [FFmpeg-devel] [PATCH 2/2] avformat/framehash: add sidedata checksum

On Wed, Apr 13, 2016 at 01:54:26AM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > Once these two patches go in I'll send one enabling version 2 output > by default. > > libavformat/hashenc.c | 36 +--- > 1 file changed, 33

Re: [FFmpeg-devel] [PATCH] avformat/framehash: enable new output

On Wed, Apr 13, 2016 at 02:22:16PM -0300, James Almer wrote: > Also, since it's compatible with version 1, make it part of it instead of the > new version 2. > > Signed-off-by: James Almer LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] avformat/dump: Fix sign bug in reported "start" time

On Tue, Apr 12, 2016 at 09:49:27PM -0700, Bryan Huh wrote: > Previously, the bug was that if -1 < start_time < 0, the reported > "start" time would lose the negative-sign. > --- > libavformat/dump.c | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) applied thanks [...] --

Re: [FFmpeg-devel] [PATCH v5 2/3] avformat/tee: Fix leaks in tee muxer when open_slave fails

On Tue, 12 Apr 2016, sebechlebsky...@gmail.com wrote: From: Jan Sebechlebsky Calling close_slave in case error is to be returned from open_slave will free allocated resources. Since failure can happen before bsfs array is initialized, close_slave must check that

Re: [FFmpeg-devel] [PATCH v5 1/3] avformat/tee: Refactor close_slaves function in tee muxer

On Tue, 12 Apr 2016, sebechlebsky...@gmail.com wrote: From: Jan Sebechlebsky Closing single slave operation is pulled out into separate function close_slave(TeeSlave*). Both close_slave and close_slaves function are moved before open_slave function. Pushed,

[FFmpeg-devel] [PATCH] avformat/hashenc: simplify hash_write_trailer

Signed-off-by: James Almer --- libavformat/hashenc.c | 28 +--- 1 file changed, 5 insertions(+), 23 deletions(-) diff --git a/libavformat/hashenc.c b/libavformat/hashenc.c index ced2f66..01b00b5 100644 --- a/libavformat/hashenc.c +++