Re: [FFmpeg-devel] [PATCH 10/12] avcodec: add MediaCodec encoder

2022-10-26 Thread Tomas Härdin
mån 2022-10-24 klockan 11:16 +0800 skrev Zhao Zhili: > > +typedef struct MediaCodecEncContext { > +    AVClass *avclass; > +    FFAMediaCodec *codec; > +    int use_ndk_codec; > +    FFANativeWindow *window; > + > +    int fps; > +    int width; > +    int height; > + > +    uint8_t *extradata; >

Re: [FFmpeg-devel] [PATCH 10/12] avcodec: add MediaCodec encoder

2022-10-27 Thread Tomas Härdin
tor 2022-10-27 klockan 10:49 +0800 skrev zhilizhao(赵志立): > > > > On Oct 27, 2022, at 05:17, Tomas Härdin wrote: > > > > mån 2022-10-24 klockan 11:16 +0800 skrev Zhao Zhili: > > > > > > +typedef struct MediaCodecEncContext { > > > +  

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mxfdec: Reduce overflows in essence_length computation

2022-11-16 Thread Tomas Härdin
sön 2022-11-13 klockan 00:44 +0100 skrev Michael Niedermayer: > Fixes: signed integer overflow: -3741319169 - 9223372036823449370 > cannot be represented in type 'long' > Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_MXF_fuzzer- > 513039428681728 > > Found-by: continuous fuzzing process

Re: [FFmpeg-devel] Developer meeting Dec 2

2022-11-24 Thread Tomas Härdin
sön 2022-11-20 klockan 17:06 +0100 skrev Jean-Baptiste Kempf: > Hello Guys & Girls, > > Some of us are planning to meet this 2nd of December, in Barcelona. > > There is no predefinite agenda, just meet, discuss and code around > FFmpeg. > Everybody that feels part of this community is invited. >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/jpeg2000dec: Move decoder structs to header files

2022-12-02 Thread Tomas Härdin
fre 2022-12-02 klockan 21:10 +0300 skrev etemesica...@gmail.com: > From: caleb Speak of the devil :) > > --- >  libavcodec/jpeg2000dec.c |  96 + >  libavcodec/jpeg2000dec.h | 126 > +++ >  2 files changed, 127 insertions(+), 95 dele

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/jpeg2000dec: Add support for HTJ2K decoding.

2022-12-02 Thread Tomas Härdin
fre 2022-12-02 klockan 21:11 +0300 skrev etemesica...@gmail.com: > > +/** > + *  Given a precomputed c, checks whether n % d == 0 > + */ > +static av_always_inline uint32_t is_divisible(uint32_t n, uint64_t > c) > +{ > +    return n * c <= c - 1; > +} This looks like something that could go in la

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec: read source timecode from pulldown component

2014-10-25 Thread Tomas Härdin
On Fri, 2014-10-24 at 17:31 -0700, Mark Reid wrote: > --- > libavformat/mxf.h | 1 + > libavformat/mxfdec.c | 31 +-- > tests/ref/lavf/mxf | 6 +++--- > tests/ref/lavf/mxf_d10 | 2 +- > 4 files changed, 34 insertions(+), 6 deletions(-) > > diff --git a/li

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: AVC Intra support

2014-10-27 Thread Tomas Härdin
On Mon, 2014-10-27 at 17:21 +, Thomas Mundt wrote: > Hi, I´ve seen that there has been approach last month to implement AVC Intra > mxf muxing. I tested the patches, but it didn´t work with any of my samples. > Since there also has been discussions about the gpl restriction, I rewrote > the

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: refactored resolving timecode component

2014-11-14 Thread Tomas Härdin
On Wed, 2014-11-12 at 12:15 -0800, Mark Reid wrote: > --- > libavformat/mxfdec.c | 36 +++- > 1 file changed, 23 insertions(+), 13 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index b533e2a..87f1e51 100644 > --- a/libavformat/mxfdec.c

Re: [FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-21 Thread Tomas Härdin
On Sun, 2014-11-16 at 02:03 +0100, Carl Eugen Hoyos wrote: > On Saturday 15 November 2014 11:57:00 pm Michael Niedermayer wrote: > > On Sat, Nov 15, 2014 at 02:50:38AM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4107 for me. > > > An alternative would be to for

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: export source package uids and names as metadata

2014-11-21 Thread Tomas Härdin
On Tue, 2014-11-18 at 16:52 -0800, Mark Reid wrote: > --- > libavformat/mxfdec.c | 74 > +++- > 1 file changed, 39 insertions(+), 35 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index fa0a2f4..8c13c24 100644 > --- a/l

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: initial support for EssenceGroups

2014-11-28 Thread Tomas Härdin
On Tue, 2014-11-25 at 15:14 -0800, Mark Reid wrote: > --- > libavformat/mxf.c | 1 + > libavformat/mxf.h | 1 + > libavformat/mxfdec.c | 148 > - > tests/ref/lavf/mxf | 6 +- > tests/ref/lavf/mxf_d10 | 2 +- > 5 files changed

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c refactor resolving MultiDescriptor and remove essence group hack

2014-12-04 Thread Tomas Härdin
On Tue, 2014-12-02 at 16:43 -0800, Mark Reid wrote: > I think this is a better way to deal with single frame essence data then my > previous way. > > --- > libavformat/mxfdec.c | 62 > > 1 file changed, 38 insertions(+), 24 deletions(-) > > d

Re: [FFmpeg-devel] [PATCH]Fix leak reading invalid mxf files

2014-12-12 Thread Tomas Härdin
On Wed, 2014-12-10 at 11:30 +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4173 for me. > To be split in two parts. > > Please comment, Carl Eugen Looks alright. Maybe you want to pass it a MXFMetadataSet** so you can use av_freep() like before? But I suppose it doesn't ma

Re: [FFmpeg-devel] [PATCH]Fix leak reading invalid mxf files

2014-12-14 Thread Tomas Härdin
On Sat, 2014-12-13 at 13:18 +0100, Carl Eugen Hoyos wrote: > On Friday 12 December 2014 01:43:19 pm Tomas Härdin wrote: > > On Wed, 2014-12-10 at 11:30 +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4173 for me. > > >

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: support demuxing opatom audio without index

2014-12-27 Thread Tomas Härdin
On Sun, 2014-12-21 at 17:46 -0800, Mark Reid wrote: > hi, > > Media Composer generates audio OPAtom mxf files that don't have index > segments. > All the uncompressed pcm audio essence data contained in a single KLV packet. > > This is my initial attempt to get demuxing working, I'm not too fami

Re: [FFmpeg-devel] [PATCH v4] libavformat/mxfdec.c: support demuxing opatom audio without index

2015-01-18 Thread Tomas Härdin
On Wed, 2015-01-14 at 19:26 -0800, Mark Reid wrote: > changes since v3: > * return if there isn’t exactly one partition > * cosmetic and gcc cleanups > * added comment about EditUnitByteCount > > --- > libavformat/mxfdec.c | 59 > > 1 file cha

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Add color siting element

2015-05-21 Thread Tomas Härdin
On Tue, 2015-05-19 at 15:35 +0100, tim nicholson wrote: > On 19/05/15 14:11, Michael Niedermayer wrote: > > On Tue, May 19, 2015 at 12:07:24PM +0100, tim nicholson wrote: > >> On 19/05/15 01:33, Michael Niedermayer wrote: > >>> The default is assumed to be 0xFF, which is what the 2009 spec lists >

Re: [FFmpeg-devel] [PATCH]lavf/mxf: Map codec_tag for Avid files if everything else fails

2015-07-11 Thread Tomas Härdin
On Fri, 2015-07-10 at 20:11 +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows decoding of the sample provided on ffmpeg-user: > http://ffmpeg.org/pipermail/ffmpeg-user/2015-July/027472.html > > Better suggestions welcome, Carl Eugen Just a quick review since I have to bounce: > +co

Re: [FFmpeg-devel] [PATCH] mxfdec: calculate the index in display order

2015-07-14 Thread Tomas Härdin
On Sat, 2015-07-11 at 18:54 +0200, Marton Balint wrote: > This should fix seeking for open GOP files as well. > > Signed-off-by: Marton Balint > --- > @@ -1411,8 +1417,7 @@ static int mxf_compute_ptses_fake_index(MXFContext > *mxf, MXFIndexTable *index_ta > break; >

Re: [FFmpeg-devel] [PATCH]lavf/mxf: Map codec_tag for Avid files if everything else fails

2015-07-21 Thread Tomas Härdin
On Fri, 2015-07-17 at 12:36 +0200, Carl Eugen Hoyos wrote: > On Saturday 11 July 2015 04:13:52 pm Tomas Härdin wrote: > > Just a quick review since I have to bounce: > > > +const MXFCodecUL ff_mxf_codec_tag_uls[] = { > > > > Haven't we moved this to mxf.c alrea

Re: [FFmpeg-devel] [PATCH] mxfdec: set AVFMT_SEEK_TO_PTS demuxer flag

2015-08-10 Thread Tomas Härdin
On Sun, 2015-08-09 at 20:32 +0200, Marton Balint wrote: > Since 53f2ef2c4afb1d49a679dea9163cb0e4671f3117 seeking is done using PTS. > > Signed-off-by: Marton Balint > --- > libavformat/mxfdec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c

Re: [FFmpeg-devel] [PATCH] mxfdec: set AVFMT_SEEK_TO_PTS demuxer flag

2015-08-14 Thread Tomas Härdin
On Mon, 2015-08-10 at 10:14 +0200, Tomas Härdin wrote: > On Sun, 2015-08-09 at 20:32 +0200, Marton Balint wrote: > > Since 53f2ef2c4afb1d49a679dea9163cb0e4671f3117 seeking is done using PTS. > > > > Signed-off-by: Marton Balint > > --- > > libavformat/mxfd

Re: [FFmpeg-devel] [PATCH] mxfdec: set AVFMT_SEEK_TO_PTS demuxer flag

2015-08-25 Thread Tomas Härdin
On Tue, 2015-08-25 at 07:58 +0100, tim nicholson wrote: > On 14/08/15 13:27, Tomas Härdin wrote: > > On Mon, 2015-08-10 at 10:14 +0200, Tomas Härdin wrote: > >> On Sun, 2015-08-09 at 20:32 +0200, Marton Balint wrote: > >>> Since 53f2ef2c4afb1d49a679dea9163cb0e4671f311

Re: [FFmpeg-devel] [PATCH]Support more mxf files with codec_ul

2015-09-02 Thread Tomas Härdin
On Tue, 2015-09-01 at 23:53 +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes decoding the sample from ticket #4820 for me. > > Please comment, Carl Eugen > -static const MXFCodecUL mxf_codec_uls[] = { > -{ { > 0x06,0x0E,0x2B,0x34,0x04,0x01,0x01,0x07,0x04,0x01,0x02,0x02,0x03,0x0

Re: [FFmpeg-devel] [RFC] avformat/mxfenc: stop encoding if unfilled video packet

2015-09-19 Thread Tomas Härdin
On Wed, 2015-09-16 at 14:33 +0200, Tobias Rapp wrote: > Hi, > > attached patch fixes ticket #4759 but I guess it is a bit too hasty to > always abort transcoding if a single frame cannot be written. I guess it > would be better to check for some "exit_on_error" like flag set but > couldn't find

Re: [FFmpeg-devel] [RFC] avformat/mxfenc: stop encoding if unfilled video packet

2015-10-04 Thread Tomas Härdin
On Mon, 2015-09-28 at 15:11 +0200, Tobias Rapp wrote: > On 19.09.2015 22:49, Tomas Härdin wrote: > > On Wed, 2015-09-16 at 14:33 +0200, Tobias Rapp wrote: > >> Hi, > >> > >> attached patch fixes ticket #4759 but I guess it is a bit too hasty to > >&g

Re: [FFmpeg-devel] [RFC] avformat/mxfenc: stop encoding if unfilled video packet

2015-10-12 Thread Tomas Härdin
On Mon, 2015-10-05 at 14:25 +0200, Tobias Rapp wrote: > On 05.10.2015 09:10, tim nicholson wrote: > > On 04/10/15 13:07, Tomas Härdin wrote: > >> On Mon, 2015-09-28 at 15:11 +0200, Tobias Rapp wrote: > >>> [...] > >> > >> For me the most importan

Re: [FFmpeg-devel] [PATCH 1/2] libavformat/mxfenc.c: Fix segfault when writing an audio packet when there has not been a video one.

2015-10-19 Thread Tomas Härdin
On Mon, 2015-10-19 at 11:40 +0200, Alexis Ballier wrote: > On Mon, 19 Oct 2015 10:30:00 +0200 > Michael Niedermayer wrote: > > > On Fri, Oct 16, 2015 at 10:42:32AM +0200, Alexis Ballier wrote: > > > This happens when writing the trailer of a file containing audio > > > but that has not muxed any

Re: [FFmpeg-devel] [PATCH 1/2] libavformat/mxfenc.c: Fix segfault when writing an audio packet when there has not been a video one.

2015-10-21 Thread Tomas Härdin
On Tue, 2015-10-20 at 16:43 +0200, Marton Balint wrote: > On Mon, 19 Oct 2015, Tomas Härdin wrote: > > > On Mon, 2015-10-19 at 11:40 +0200, Alexis Ballier wrote: > >> On Mon, 19 Oct 2015 10:30:00 +0200 > >> Michael Niedermayer wrote: > >> > >>&g

Re: [FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-21 Thread Tomas Härdin
On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > klv_decode_ber_length cannot return -1, but can return AVERROR_INVALIDDATA. > Store its return value in a signed integer (instead of unsigned > KLVPacket.length) and forward the error appropriately. > --- > libavformat/mxfdec.c | 6 --

Re: [FFmpeg-devel] [PATCH 02/11] libavformat/mxfdec.c: cosmetics: Add missing space after '?' in log message.

2015-10-21 Thread Tomas Härdin
On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > --- > libavformat/mxfdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 94a953b..0ae7ce6 100644 > --- a/libavformat/mxfdec.c > +++ b/libavformat/mxfdec.c > @@

Re: [FFmpeg-devel] [PATCH 03/11] libavformat/mxfdec.c: Report dark metadata keys only when they match no parser at all, not everytime they fail to match one.

2015-10-21 Thread Tomas Härdin
On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > --- > libavformat/mxfdec.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 0ae7ce6..593604e 100644 > --- a/libavformat/mxfdec.c > +++ b/libavformat/mxfde

Re: [FFmpeg-devel] [PATCH 04/11] libavformat/mxfdec.c: Try to increment current edit before rejecting a klv that spans onto next edit unit.

2015-10-21 Thread Tomas Härdin
On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > Some files such as those from tickets #2817 & #2776 claim to have constant > edit unit size but, > in fact, have some of them that are smaller. This confuses the demuxer that > tries to infer the > current edit unit from the position in t

Re: [FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-24 Thread Tomas Härdin
On Thu, 2015-10-22 at 19:25 +0200, Alexis Ballier wrote: > On Wed, 21 Oct 2015 23:31:29 +0200 > Tomas Härdin wrote: > > > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > > klv_decode_ber_length cannot return -1, but can return > > > AVERROR_INVALID

Re: [FFmpeg-devel] [PATCH 04/11] libavformat/mxfdec.c: Try to increment current edit before rejecting a klv that spans onto next edit unit.

2015-10-25 Thread Tomas Härdin
On Thu, 2015-10-22 at 19:47 +0200, Alexis Ballier wrote: > On Wed, 21 Oct 2015 23:45:07 +0200 > Tomas Härdin wrote: > > > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > > Some files such as those from tickets #2817 & #2776 claim to have > > >

Re: [FFmpeg-devel] [PATCH 04/11] libavformat/mxfdec.c: Try to increment current edit before rejecting a klv that spans onto next edit unit.

2015-11-06 Thread Tomas Härdin
On Sun, 2015-10-25 at 21:43 +0100, Tomas Härdin wrote: > On Thu, 2015-10-22 at 19:47 +0200, Alexis Ballier wrote: > > On Wed, 21 Oct 2015 23:45:07 +0200 > > Tomas Härdin wrote: > > > > > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > > > S

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Set width to actual coded_width for AVCI50

2015-11-24 Thread Tomas Härdin
On Tue, 2015-11-24 at 12:02 +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5029. > > Please comment, Carl Eugen Looks simple enough, but (ab)using MXFCodecUL like that has a slight stink to it. Can we perhaps rename the struct to something that signals its more general use

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Set width to actual coded_width for AVCI50

2015-11-27 Thread Tomas Härdin
On Thu, 2015-11-26 at 00:48 +0100, Carl Eugen Hoyos wrote: > On Wednesday 25 November 2015 12:57:07 am Tomas Härdin wrote: > > On Tue, 2015-11-24 at 12:02 +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #5029. > > > &

Re: [FFmpeg-devel] [PATCH] lavf/mxfdec: fix seeking before the first keyframe

2015-12-07 Thread Tomas Härdin
On Sat, 2015-11-28 at 02:56 +0100, Marton Balint wrote: > Regression since 53f2ef2c4afb1d49a679dea9163cb0e4671f3117. > Fixes ticket #5017. > > Signed-off-by: Marton Balint > --- > libavformat/mxfdec.c | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/libavformat/mxfdec.c b/li

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Fix integer overflow in length computation

2015-12-14 Thread Tomas Härdin
On Wed, 2015-12-09 at 18:13 +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes: CID1341577 > > Signed-off-by: Michael Niedermayer > --- > libavformat/mxfenc.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavformat/mxfenc.c b/libavforma

Re: [FFmpeg-devel] [PATCH] Several, actually, regarding the QuickTime palette issue in Matroska

2015-12-16 Thread Tomas Härdin
On Wed, 2015-12-16 at 20:30 +0100, Mats Peterson wrote: > On 12/16/2015 08:27 PM, Mats Peterson wrote: > > On 12/16/2015 08:26 PM, Kieran Kunhya wrote: > >>> I have said before that I'm not going to dwell into the complexities > >>> of Git, > >>> since it feels about as wasteful as programming a GU

Re: [FFmpeg-devel] [PATCH 5/6] libavformat/mxf: add dnxhr codec ul

2016-08-07 Thread Tomas Härdin
On Mon, 2016-07-04 at 18:07 -0700, Mark Reid wrote: > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 0affca9..8f2f10a 100644 > --- a/libavformat/mxfdec.c > +++ b/libavformat/mxfdec.c > @@ -1098,6 +1098,10 @@ static int mxf_match_uid(const UID key, const UID uid, > int len) >  sta

[FFmpeg-devel] New MXF maintainer wanted

2016-08-20 Thread Tomas Härdin
Hi Long story short: I no longer work with MXF (or broadcast in general), and my free time is currently being consumed by other projects. Therefore I feel it'd be better if someone took over the role as MXF maintainer. I took a few shots at the recent MXF tickets on trac, but I've forgotten too m

Re: [FFmpeg-devel] [RFC]lavf/mxfdec: Assume first track if track number is unknown

2016-03-13 Thread Tomas Härdin
On Thu, 2016-03-10 at 15:06 +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5312 here. > The OP claims that the file plays fine with Mainconcept softare,  > afaict the track number of the video track in the mxf header  > (0x15010800) does not match the track number of the vid

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Fix canopus essence element size

2016-03-13 Thread Tomas Härdin
On Thu, 2016-03-10 at 15:08 +0100, Carl Eugen Hoyos wrote: > Hi! > > While debugging ticket #5312, I realized that I included the  > track number in the Canopus essence element. OK /Tomas signature.asc Description: This is a digitally signed message part ___

Re: [FFmpeg-devel] Support AS-10 MXF

2016-04-04 Thread Tomas Härdin
On Tue, 2016-03-29 at 22:29 +0200, NabFab wrote: > Hi, > Could you please tell me if ffmpeg is supporting AS-10 or AS-11 MXF  > format according to AMWA specifications ? > Thank you Do you mean muxing or demuxing? I expect mxfdec should be perfectly capable of demuxing. It's been a while since I

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: check if source_package is NULL

2016-06-11 Thread Tomas Härdin
On Tue, 2016-05-31 at 23:02 +0200, Marton Balint wrote: > Fixes ticket #5554. > > Signed-off-by: Marton Balint > --- >  libavformat/mxfdec.c | 2 +- >  1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index f8cf922..9bf676c 100644 > ---

Re: [FFmpeg-devel] [PATCHv2] Revert "do not write f2 if not interlaced"

2016-02-01 Thread Tomas Härdin
On Fri, 2016-01-29 at 17:45 +0100, Sebastian Dröge wrote: > From: Sebastian Dröge > > This reverts commit 8ed82d8174a666f80ab8834e3617cbe91ae740a9. > > SMPTE S377-1-2009c defines in F.4.1 that the Video Line Map should > always be an array with two 32 bit integers as elements. This is > repeated

Re: [FFmpeg-devel] [PATCH 09/12] avformat/mxfenc: use ff_parse_creation_time_metadata

2016-02-16 Thread Tomas Härdin
On Sat, 2016-02-06 at 20:13 +0100, Marton Balint wrote: > Signed-off-by: Marton Balint > --- >  libavformat/mxfenc.c | 5 + >  1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c > index 6da8b10..cd13f89 100644 > --- a/libavformat/mxfenc.

Re: [FFmpeg-devel] [PATCH]lavf/mxfdec: Support canopus codecs

2016-03-09 Thread Tomas Härdin
On Wed, 2016-03-09 at 15:21 +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #5316 here. > I unfortunately have no idea what the hunk in mxf_read_header()  > does, decoding works fine without it. Strange. Why was mxf_canopus_essence_element_key added then? Pushing without it i

Re: [FFmpeg-devel] [PATCH v5] libavformat/mxfenc: write package name metadata

2015-03-05 Thread Tomas Härdin
On Wed, 2015-03-04 at 12:31 -0800, Mark Reid wrote: > +/* > + * Returns the calculated length a local tag containing an utf-8 string as > utf-16 > + */ > +static int mxf_utf16_local_tag_length(const char *utf8_str) > +{ > +uint64_t size; > + > +if (!utf8_str) > +return 0; > + > +

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Detect XYZ pixel format for digital cinema files

2015-03-05 Thread Tomas Härdin
On Thu, 2015-03-05 at 00:36 +0200, Vilius Grigaliūnas wrote: > While the native jpeg2000 decoder can determine pixel format correctly > from the codestream, libopenjpeg wrapper cannot. To make sure that > the output is correct when using libopenjpeg to decode digital > cinema files, we do detection

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec: export user comments metadata

2015-03-13 Thread Tomas Härdin
On Fri, 2015-03-06 at 13:24 -0800, Mark Reid wrote: > +static int mxf_read_indirect_value(void *arg, AVIOContext *pb, int size) > +{ > +MXFTaggedValue *tagged_value = arg; > +uint8_t key[17]; > + > +if (size <= 17) > +return 0; > + > +avio_read(pb, key, 17); Really Avid, 17

Re: [FFmpeg-devel] [PATCH] mxfenc: ensure mxf->body_partition_offset is not NULL before using it

2015-03-13 Thread Tomas Härdin
On Thu, 2015-03-12 at 17:48 +0100, Andreas Cadhalpun wrote: > This fixes a crash, when trying to mux h264 into mxf_opatom. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/mxfenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfenc.c b/libavforma

Re: [FFmpeg-devel] [PATCH] libavformat/mxfenc: write user comment metadata

2015-03-15 Thread Tomas Härdin
On Sat, 2015-03-14 at 17:59 -0700, Mark Reid wrote: > --- > libavformat/mxfenc.c | 66 > +-- > tests/ref/lavf/mxf| 6 ++--- > tests/ref/lavf/mxf_d10| 2 +- > tests/ref/lavf/mxf_opatom | 2 +- > 4 files changed, 69 insertions(+), 7 de

Re: [FFmpeg-devel] [PATCH 1/2] libavformat/mxfenc: add container duration and package name to primer pack

2015-03-23 Thread Tomas Härdin
On Sat, 2015-03-21 at 16:45 -0700, Mark Reid wrote: > --- > libavformat/mxfenc.c | 2 ++ > tests/ref/lavf/mxf | 6 ++--- > tests/ref/lavf/mxf_d10 | 2 +- > tests/ref/lavf/mxf_opatom | 4 ++-- > tests/ref/seek/lavf-mxf_opatom | 54 > +-

Re: [FFmpeg-devel] [PATCH 2/2] libavformat/mxfenc: add support for muxing mxf opatom audio

2015-03-23 Thread Tomas Härdin
On Sat, 2015-03-21 at 16:45 -0700, Mark Reid wrote: > --- > libavformat/mxfenc.c | 100 > ++- > 1 file changed, 83 insertions(+), 17 deletions(-) Looks fine as far as I can tell. My only nitpick is that it might be better to call "timecode rate" Ed

Re: [FFmpeg-devel] [PATCH 2/2] libavformat/mxfenc: add support for muxing mxf opatom audio

2015-03-27 Thread Tomas Härdin
On Wed, 2015-03-25 at 15:43 -0700, Mark Reid wrote: > On Mar 24, 2015 7:05 AM, "Tomas Härdin" wrote: > > > > On Sat, 2015-03-21 at 16:45 -0700, Mark Reid wrote: > > > --- > > > libavformat/mxfenc.c | 100 > ++

Re: [FFmpeg-devel] [PATCH v2] libavformat/mxfenc: added support mxf opatom audio muxing

2015-04-11 Thread Tomas Härdin
On Fri, 2015-04-10 at 20:02 -0700, Mark Reid wrote: > @@ -2055,8 +2083,35 @@ static int mxf_write_header(AVFormatContext *s) > av_log(s, AV_LOG_ERROR, "MXF D-10 only support 16 or 24 > bits le audio\n"); > } > sc->index = > ((MXFStreamContext

Re: [FFmpeg-devel] [PATCH]Set interlaced flag when muxing interlaced dnxhd into mxf

2015-04-19 Thread Tomas Härdin
On Fri, 2015-04-17 at 00:31 +0200, Carl Eugen Hoyos wrote: > Hi! > > Completely untested. > > Please review, Carl Eugen No idea really. Can't this flag be taken automagically from the codec somehow? /Tomas signature.asc Description: This is a digitally signed message part

Re: [FFmpeg-devel] [PATCH v2] added ULs for demuxing avid media composer mxf files

2014-08-14 Thread Tomas Härdin
On Tue, 2014-08-12 at 14:58 -0700, Mark Reid wrote: > UL values copied from FMbc version of mxf.c > Fixes Ticket#1554, Ticket#3100 and Ticket#3450 > --- > libavformat/mxf.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavformat/mxf.c b/libavformat/mxf.c > index d20ed94..8ebfdf2 1

Re: [FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-09-10 Thread Tomas Härdin
On Tue, 2014-08-19 at 22:30 +0200, Michael Niedermayer wrote: > On Tue, Aug 19, 2014 at 01:30:24AM +0200, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch removes a request for samples of which we already > > have several that all work fine. > > field_dominance can have 256 different values

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-10 Thread Tomas Härdin
On Tue, 2014-09-09 at 12:37 +0200, Michael Niedermayer wrote: > On Mon, Sep 08, 2014 at 12:17:14PM +, Gaullier Nicolas wrote: > > I did not found an easy way to set up initialization values to properly > > handle defaults but I am not a highly skilled developer, and maybe someone > > will fin

Re: [FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-09-24 Thread Tomas Härdin
On Tue, 2014-09-16 at 09:03 +0100, tim nicholson wrote: > On 10/09/14 21:45, Tomas Härdin wrote: > > On Tue, 2014-08-19 at 22:30 +0200, Michael Niedermayer wrote: > >> On Tue, Aug 19, 2014 at 01:30:24AM +0200, Carl Eugen Hoyos wrote: > >>> Hi! > >>>

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mxfenc: H.264 Intra support

2014-09-24 Thread Tomas Härdin
On Sat, 2014-09-13 at 12:36 +0200, Michael Niedermayer wrote: > From: Baptiste Coudurier > > Ported by michael from ffmbc to ffmpeg > the code is under CONFIG_GPL as ffmbc is GPL > > Signed-off-by: Michael Niedermayer > --- > libavformat/mxfenc.c | 143 > +

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: read reel_name and source timecode from physical source package

2014-09-28 Thread Tomas Härdin
On Thu, 2014-09-25 at 16:13 -0700, Mark Reid wrote: > --- > libavformat/mxfdec.c | 118 > ++- > 1 file changed, 97 insertions(+), 21 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 7a4633f..3a1889f 100644 > --- a/l

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: add jpeg2000 support

2014-10-02 Thread Tomas Härdin
On Tue, 2014-09-30 at 15:37 +0200, Benoit Fouet wrote: > Hi, > > this patch adds support for j2k muxing in MXF. > tested with: > $ ffmpeg -t 5 -f lavfi -i testsrc -y -c:v libopenjpeg -y out.mxf > > Played back in ffplay (linux), vlc (windows), Acrok MXF converter (windows). > I have no idea again

Re: [FFmpeg-devel] [PATCH v2] avformat/mxfdec: read reel_name and source timecode from physical source package

2014-10-06 Thread Tomas Härdin
On Sat, 2014-10-04 at 17:42 -0700, Mark Reid wrote: > --- > libavformat/mxfdec.c | 120 > ++- > 1 file changed, 99 insertions(+), 21 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 7a4633f..ef4c4ec 100644 > --- a/l

Re: [FFmpeg-devel] [PATCH] mxfdec: minor simplification.

2014-10-19 Thread Tomas Härdin
On Sat, 2014-10-18 at 15:26 +0200, Reimar Döffinger wrote: > Signed-off-by: Reimar Döffinger > --- > libavformat/mxfdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 9306cc6..f916d01 100644 > --- a/libavformat/mxfde

Re: [FFmpeg-devel] [PATCH v1] avformat/mxfdec: use av_asprintf()

2020-03-23 Thread Tomas Härdin
sön 2020-03-22 klockan 23:03 +0800 skrev lance.lmw...@gmail.com: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/mxfdec.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index 9113e2a09c..3374f36a

Re: [FFmpeg-devel] [PATCH] avformat/mxfdec: Correct confusing struct tag

2020-03-29 Thread Tomas Härdin
sön 2020-03-29 klockan 06:06 +0200 skrev Andreas Rheinhardt: > Don't use typedef struct MXFTrack {...} MXFTimecodeComponent, in > particular given the fact that MXFTrack is a type of its own. > > Signed-off-by: Andreas Rheinhardt > --- > libavformat/mxfdec.c | 2 +- > 1 file changed, 1 insertion

Re: [FFmpeg-devel] [PATCH 2/5] avformat/mxfdec: reindent code

2020-05-04 Thread Tomas Härdin
tor 2020-04-30 klockan 20:59 +0800 skrev lance.lmw...@gmail.com: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/mxfdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index fdd0dd2a88..02a2a6d97a 100

Re: [FFmpeg-devel] [PATCH v4 7/8] avformat: implement retiming directly in mxfenc and gxfenc

2020-05-04 Thread Tomas Härdin
tis 2020-04-28 klockan 19:37 +0200 skrev Marton Balint: > Generic retime functionality is replaced by a few lines of code directly in > the > muxers which used it, which seems a lot easier to understand and this way the > retiming is not dependant of the input durations. > > Signed-off-by: Marton

Re: [FFmpeg-devel] [PATCH v4 7/8] avformat: implement retiming directly in mxfenc and gxfenc

2020-05-04 Thread Tomas Härdin
tis 2020-04-28 klockan 19:37 +0200 skrev Marton Balint: > Generic retime functionality is replaced by a few lines of code > directly in the > muxers which used it, which seems a lot easier to understand and this > way the > retiming is not dependant of the input durations. > > Signed-off-by: Marto

Re: [FFmpeg-devel] [PATCH 2/5] avformat/mxfdec: reindent code

2020-05-04 Thread Tomas Härdin
mån 2020-05-04 klockan 18:58 +0800 skrev Lance Wang: > On Mon, May 4, 2020 at 4:13 PM Tomas Härdin > wrote: > > > tor 2020-04-30 klockan 20:59 +0800 skrev lance.lmw...@gmail.com: > > > From: Limin Wang > > > > > > Signed-off-by: Limin Wang > > &

Re: [FFmpeg-devel] [RFC]separation of multiple outputs' encoding

2020-05-19 Thread Tomas Härdin
fre 2020-05-15 klockan 10:14 +0800 skrev Tao Zhang: > Marton Balint 于2020年5月15日周五 上午2:33写道: > > > > > > On Thu, 14 May 2020, Tao Zhang wrote: > > > > > Hi, > > > FFmpeg supports multiple outputs created out of the same input in the > > > same process like > > > ffmpeg -i input -filter_complex '

Re: [FFmpeg-devel] [PATCH/RFC] libavcodec/cinepak: Separate decoding from parsing

2019-09-02 Thread Tomas Härdin
sön 2019-09-01 klockan 23:07 +0200 skrev Carl Eugen Hoyos: > Am So., 1. Sept. 2019 um 22:58 Uhr schrieb Tomas Härdin < > tjop...@acc.umu.se>: > > > Attached patch separates parsing from decoding in the Cinepak decoder. > > It puts in some rather strict checks whic

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/apedec: Fix 32bit int overflow in do_apply_filter()

2019-09-03 Thread Tomas Härdin
tis 2019-09-03 klockan 02:14 +0200 skrev Michael Niedermayer: > Fixes: signed integer overflow: 2147480546 + 4096 cannot be represented in > type 'int' > Fixes: > 16280/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_APE_fuzzer-5123442566758400 > > Found-by: continuous fuzzing process > https

Re: [FFmpeg-devel] [PATCH 3/5] avcodec: add max_samples

2019-09-03 Thread Tomas Härdin
tis 2019-09-03 klockan 09:20 +0200 skrev Paul B Mahol: > > On 9/3/19, Michael Niedermayer wrote: > > TODO: APIChanges, bump version > > > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/avcodec.h | 8 > > libavcodec/options_table.h | 1 + > > test

Re: [FFmpeg-devel] [PATCH 1/5] avcodec/apedec: Fix 32bit int overflow in do_apply_filter()

2019-09-03 Thread Tomas Härdin
tis 2019-09-03 klockan 13:25 +0200 skrev Michael Niedermayer: > On Tue, Sep 03, 2019 at 12:16:47PM +0200, Tomas Härdin wrote: > > tis 2019-09-03 klockan 02:14 +0200 skrev Michael Niedermayer: > > > Fixes: signed integer overflow: 2147480546 + 4096 cannot be represented

Re: [FFmpeg-devel] [PATCH 3/5] avcodec: add max_samples

2019-09-03 Thread Tomas Härdin
tis 2019-09-03 klockan 13:10 +0200 skrev Michael Niedermayer: > On Tue, Sep 03, 2019 at 12:21:26PM +0200, Tomas Härdin wrote: > > tis 2019-09-03 klockan 09:20 +0200 skrev Paul B Mahol: > > > On 9/3/19, Michael Niedermayer wrote: > > > > TODO: APIChanges, bump versi

Re: [FFmpeg-devel] [PATCH/RFC] libavcodec/cinepak: Separate decoding from parsing

2019-09-03 Thread Tomas Härdin
tis 2019-09-03 klockan 13:53 +0200 skrev Michael Niedermayer: > On Sun, Sep 01, 2019 at 10:58:27PM +0200, Tomas Härdin wrote: > > libavcodec/cinepak.c| 337 --- > > - > > tests/ref/fate/cvid-palette |1 > > tests/r

Re: [FFmpeg-devel] [PATCH v2] avformat/mxfenc: fix warning: unused function 'klv_ber_length' [-Wunused-function]

2019-09-04 Thread Tomas Härdin
ons 2019-09-04 klockan 22:27 +0800 skrev Limin Wang: > On Fri, Aug 30, 2019 at 08:42:08AM +0200, Tomas Härdin wrote: > > tor 2019-08-29 klockan 16:44 +0800 skrev lance.lmw...@gmail.com: > > > From: Limin Wang > > > > > > Signed-off-by: Limin Wang >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/g2meet: Check if adjusted pixel was on the stack

2019-09-09 Thread Tomas Härdin
mån 2019-09-09 klockan 22:16 +0200 skrev Michael Niedermayer: > This basically checks if a pixel that was coded with prediction > and residual could have been stored using a previous case. > This avoids basically a string of 0 symbols stored in less than > 50 bytes to hit a O(n²) codepath. > > Fix

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/g2meet: Check for end of input in jpg_decode_block()

2019-09-09 Thread Tomas Härdin
mån 2019-09-09 klockan 22:16 +0200 skrev Michael Niedermayer: > Fixes: Timeout (100sec -> 0.7sec) > Fixes: > 8668/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_G2M_fuzzer-5174143888130048 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpe

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/g2meet: Check for end of input in jpg_decode_block()

2019-09-11 Thread Tomas Härdin
tis 2019-09-10 klockan 16:16 +0200 skrev Michael Niedermayer: > On Mon, Sep 09, 2019 at 11:04:32PM +0200, Tomas Härdin wrote: > > mån 2019-09-09 klockan 22:16 +0200 skrev Michael Niedermayer: > > > Fixes: Timeout (100sec -> 0.7sec) > > > Fixes: > > &

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/g2meet: Check for end of input in jpg_decode_block()

2019-09-12 Thread Tomas Härdin
tor 2019-09-12 klockan 00:21 +0200 skrev Michael Niedermayer: > On Wed, Sep 11, 2019 at 11:18:47PM +0200, Tomas Härdin wrote: > > tis 2019-09-10 klockan 16:16 +0200 skrev Michael Niedermayer: > > > On Mon, Sep 09, 2019 at 11:04:32PM +0200, Tomas Härdin wrote: > > > >

Re: [FFmpeg-devel] [PATCH v1 1/4] avutil/avstring: support input path is a null pointer or empty string

2019-09-16 Thread Tomas Härdin
mån 2019-09-16 klockan 09:03 +0800 skrev lance.lmw...@gmail.com: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavutil/avstring.c | 12 > libavutil/avstring.h | 13 + > 2 files changed, 17 insertions(+), 8 deletions(-) > > diff --git a/libavutil/avstring.c

Re: [FFmpeg-devel] Is it ok to add G.722.1 decoder as external lib?

2019-09-17 Thread Tomas Härdin
tis 2019-09-17 klockan 09:29 +0900 skrev Hyun Yoo: > I implemented a g.722.1 decoder by linking FreeSwitch's libg722_1 > as external lib like libilbc, libspeex(ex. configure --enable- > libg722_1) > (https://github.com/traviscross/freeswitch/tree/master/libs/libg722_1 > ) > > But I'm not sure abou

Re: [FFmpeg-devel] [PATCH] avutil/encryption_info: Don't pass NULL to memcpy

2019-09-18 Thread Tomas Härdin
ons 2019-09-18 klockan 02:03 +0200 skrev Andreas Rheinhardt: > The pointer arguments to memcpy (and several other functions of the > C standard library) are not allowed to be NULL, not even when the number > of bytes to copy is zero. An AVEncryptionInitInfo's data pointer is > explicitly allowed to

Re: [FFmpeg-devel] [PATCH v1 1/4] avutil/avstring: support input path is a null pointer or empty string

2019-09-18 Thread Tomas Härdin
ons 2019-09-18 klockan 06:41 +0800 skrev Limin Wang: > On Tue, Sep 17, 2019 at 06:22:39PM +0200, Marton Balint wrote: > > > > On Mon, 16 Sep 2019, Tomas Härdin wrote: > > > > > mån 2019-09-16 klockan 09:03 +0800 skrev lance.lmw...@gmail.com: > > > > Fro

Re: [FFmpeg-devel] Is it ok to add G.722.1 decoder as external lib?

2019-09-19 Thread Tomas Härdin
tor 2019-09-19 klockan 13:58 +0900 skrev Hyun Yoo: > On Thu, Sep 19, 2019 at 7:28 AM Carl Eugen Hoyos wrote: > > Am Di., 17. Sept. 2019 um 02:37 Uhr schrieb Hyun Yoo > > : > > > I implemented a g.722.1 decoder by linking FreeSwitch's libg722_1 > > > as external lib like libilbc, libspeex(ex. conf

Re: [FFmpeg-devel] [PATCH v2 10/15] avcodec/cinepakenc: Fix invalid shifts

2019-09-28 Thread Tomas Härdin
lör 2019-09-28 klockan 04:26 +0200 skrev Andreas Rheinhardt: > Fixes: left shift of 1 by 31 places cannot be represented in type 'int'. > Affected the FATE-tests vsynth1-cinepak, vsynth2-cinepak and > vsynth_lena-cinepak. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/cinepakenc.c | 11

Re: [FFmpeg-devel] [PATCH 5/5] avcodec/flicvideo: Use bytestream2_get_buffer() in flic_decode_frame_15_16BPP() for FLI_COPY

2019-09-28 Thread Tomas Härdin
tor 2019-09-26 klockan 23:10 +0200 skrev Michael Niedermayer: > Fixes: Timeout(103sec -> 3sec) > Fixes: > 17678/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_FLIC_fuzzer-5715436989054976 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/g2meet: Check for end of input in jpg_decode_block()

2019-09-30 Thread Tomas Härdin
lör 2019-09-28 klockan 17:47 +0200 skrev Michael Niedermayer: > On Thu, Sep 12, 2019 at 11:09:16PM +0200, Tomas Härdin wrote: > > tor 2019-09-12 klockan 00:21 +0200 skrev Michael Niedermayer: > > > On Wed, Sep 11, 2019 at 11:18:47PM +0200, Tomas Härdin wrote: > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/zmbv: Simplify assigning decode_intra function pointer

2019-10-07 Thread Tomas Härdin
mån 2019-10-07 klockan 02:57 +0200 skrev Andreas Rheinhardt: > zmbv has only one function for decoding intra frames, namely > decode_intra. This can be used to simplify the process of choosing the > right function pointer. > > This also removes spec-incompliant conversions between function pointer

Re: [FFmpeg-devel] [PATCH] avcodec/zmbv: Simplify assigning decode_intra function pointer

2019-10-08 Thread Tomas Härdin
mån 2019-10-07 klockan 16:42 + skrev Andreas Rheinhardt: > Tomas Härdin: > > mån 2019-10-07 klockan 02:57 +0200 skrev Andreas Rheinhardt: > > > zmbv has only one function for decoding intra frames, namely > > > decode_intra. This can be used to simplify the process

Re: [FFmpeg-devel] [PATCH] avcodec/zmbvenc: Correct offset in buffer

2019-10-10 Thread Tomas Härdin
tor 2019-10-10 klockan 11:47 +0200 skrev Andreas Rheinhardt: > zmbvenc allocates a buffer for a picture with padding on all four sides: > The stride is choosen so large that it already contains padding on the > right; the height also includes padding rows. The padding on the right > of each row is

Re: [FFmpeg-devel] [PATCH 5/8] avformat/mxfdec: Clear metadata_sets_count in mxf_read_close()

2019-11-06 Thread Tomas Härdin
tor 2019-10-31 klockan 18:58 +0100 skrev Michael Niedermayer: > This avoids problems if the function is called twice > > Signed-off-by: Michael Niedermayer > --- > libavformat/mxfdec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c > index f

<    3   4   5   6   7   8   9   10   11   12   >