[FFmpeg-cvslog] r21480 - trunk/ffmpeg.c

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> On Wed, Jan 27, 2010 at 03:31:15PM +0000, M?ns Rullg?rd wrote:
>> michael <subversion at mplayerhq.hu> writes:
>> 
>> > Author: michael
>> > Date: Wed Jan 27 14:27:18 2010
>> > New Revision: 21480
>> >
>> > Log:
>> > Print "Multiple frames in a packet" warning independant of CODEC_CAP_SUBFRAMES
>> > because CODEC_CAP_SUBFRAMES is primarely misused to hide this warning otherwise
>> > which renders the flag completely useless.
>> 
>> Are you going to fix the causes of the warning?
>
> i dont have the time currently

Thanks for making it only print once anyway.  Now I can ignore it in peace.

>>  You've shot down
>> every attempt by others so far...
>
> noone attempted to fix the warnings. You commited a 1 line hack that hid the
> bugs that cause the warnings.

I was told it was the right thing to do for those formats.  I guess
they're all liars then.

> If you want to hide the warnings do it in a way that is maintainable
> like a "//" before the code printing them

You call *THAT* maintainable?!?!?!?!?

-- 
M?ns Rullg?rd
mans at mansr.com