[FFmpeg-devel] [PATCH] do not write empty udta atom
Baptiste Coudurier
baptiste.coudurier
Wed Oct 24 16:16:33 CEST 2007
Benoit Fouet wrote:
> Baptiste Coudurier wrote:
>> Benoit Fouet wrote:
>>
>>> LIBAVFORMAT_IDENT is only written in MODE_MOV ATM, and if not in
>>> bitexact mode.
>>>
>> It is also written in MODE_MP4 if any metadata is available, check
>> mov_write_ilst_tag.
>>
>>
>
> indeed, thanks
>
>>> [...]
>>> +
>>> + if (s->title[0] || s->author[0] || s->album[0] || s->year ||
>>> + s->comment[0] || s->genre[0] || s->track ||
>>> + (mov->mode == MODE_MOV &&
>>> + (mov->tracks[0].enc && !mov->tracks[0].enc->flags & CODEC_FLAG_BITEXACT) || req)) {
>>>
>> movenc.c:1238: warning: suggest parentheses around && within ||
>>
>>
>
> oops... not only the warning, but it was plain wrong :(
> updated (and i think last) patch attached.
>
Looks ok, don't forget to reindent.
--
Baptiste COUDURIER GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A. http://www.smartjog.com
Key fingerprint 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312
More information about the ffmpeg-devel
mailing list