[FFmpeg-devel] [PATCH] QCELP : warn for buf_size and bitrate mismatch only once
Kenan Gillet
kenan.gillet
Wed Dec 10 00:37:14 CET 2008
On Fri, Dec 5, 2008 at 4:09 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Thu, Dec 04, 2008 at 02:35:26PM -0800, Kenan Gillet wrote:
>> Hi,
>>
>> at the moment this warning is logged for every frame,
>> at AV_LOG_WARNING level.
>> For at least 2 samples (h263.mov, blue_earth.mov),
>> it creates an unnecessary long output.
>>
>> Should we demote it to AV_LOG_DEBUG or only show it once ?
>> or only show it once like in the patch attached?
>>
>> Kenan
>
>> Index: libavcodec/qcelpdec.c
>> ===================================================================
>> --- libavcodec/qcelpdec.c (revision 15997)
>> +++ libavcodec/qcelpdec.c (working copy)
>> @@ -69,6 +69,7 @@
>> float pitch_gain[4];
>> uint8_t pitch_lag[4];
>> uint16_t first16bits;
>> + uint16_t warn_buf_mismatch_bitrate;
>
> you need exactly 16bits to store a flag?
done
>
> also i think that warned, that is past tense or something is a better
> choice than warn to make it clear that a "true" means it has been done
> already. Instead of 1 meaning that the warning should still be printed
changed to warned_
> [...]
new patch attached
Kenan
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: qcelp-warn-buf-mismatches-bitrate-once-2.patch.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081209/09f6f5b1/attachment.txt>
More information about the ffmpeg-devel
mailing list