[FFmpeg-devel] [PATCH]Show mxf Universal Label with loglevel verbose
dave at dericed.com
Tue Feb 12 01:53:11 CET 2013
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.
More information about the ffmpeg-devel