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

2015-03-04 Thread Mark Reid
On Wed, Mar 4, 2015 at 4:07 AM, wrote: > On 2015-03-03 05:06, Mark Reid wrote: > >> +/* >> + * Returns the calculated length a local tag containing an utf-8 >> string as utf-16 >> + */ >> +static uint64_t mxf_utf16_local_tag_length(const char *utf8_str) >> +{ >> +return utf8_str? 4 + mxf_utf1

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

2015-03-04 Thread tomas . hardin
On 2015-03-03 05:06, Mark Reid wrote: +/* + * Returns the calculated length a local tag containing an utf-8 string as utf-16 + */ +static uint64_t mxf_utf16_local_tag_length(const char *utf8_str) +{ +return utf8_str? 4 + mxf_utf16len(utf8_str) * 2: 0; Should return zero if mxf_utf16len()*2

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

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