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

2015-03-05 Thread Mark Reid
> > > > if (size >= UINT16_MAX/2) { > > that makes way more sense :p, I'll send a new patch ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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; > + > +

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

2015-03-04 Thread Mark Reid
--- libavformat/mxfenc.c | 97 +-- tests/ref/lavf/mxf| 6 +-- tests/ref/lavf/mxf_d10| 2 +- tests/ref/lavf/mxf_opatom | 2 +- 4 files changed, 91 insertions(+), 16 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c

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

2015-03-04 Thread Mark Reid
changes since v4: * added UINT16_MAX check in mxf_utf16_local_tag_length and mxf_write_local_tag_utf16 Mark Reid (1): libavformat/mxfenc: write package name metadata libavformat/mxfenc.c | 97 +-- tests/ref/lavf/mxf| 6 +-- tests/ref/