[FFmpeg-devel] [PATCH]Show mxf Universal Label with loglevel verbose

[Dave Rice]

On Feb 11, 2013, at 9:53 AM, Tim Nicholson <nichot20 at yahoo.com> wrote:

> On 11/02/13 12:14, Carl Eugen Hoyos wrote:
>> Tomas Härdin <tomas.hardin <at> codemill.se> writes:
>> 
>>>>         st->codec->codec_id = (enum AVCodecID)codec_ul->id;
>>>> +        av_log(mxf->fc, AV_LOG_VERBOSE, "%s: Universal Label: 0x",
>>>> +               avcodec_get_name(st->codec->codec_id));
>>> 
>>> Do we need the "0x"?
>> 
>> Pushed without "0x".
>> If the SMPTE standard notation is needed (I don't know), 
>> it should be easy to change the code.
>> 
> 
> I think the SMPTE is a bit more readable as it helps identify the
> hierarchical tree.
> 
> Long strings of hex send me cross eyed....

I originally proposed a hex expression to keep the UL expression consistent with how ffmpeg express fourccs, but now agree that Tomas's suggestion of SMPTE's format is preferable.
Dave Rice