[FFmpeg-devel] [PATCH] libavcodec/lpc.c: Fix warning about uninitialized variable
Jai Menon
jmenon86
Tue Feb 10 10:13:20 CET 2009
On Tue, Feb 10, 2009 at 2:38 PM, Patrik Kullman <patrik at yes.nu> wrote:
>
>
> On Tue, 2009-02-10 at 14:29 +0530, Jai Menon wrote:
>> On Tue, Feb 10, 2009 at 2:14 PM, Jai Menon <jmenon86 at gmail.com> wrote:
>> > On Tue, Feb 10, 2009 at 2:08 PM, Patrik Kullman <patrik at yes.nu> wrote:
>> >> If use_lpc < 1 and max_order > 0, weight would be uninitialized.
>> >
>> > since a use_lpc of 0 was never intended, maybe an assert could be used....
>>
>> to clarify, i meant an assert on use_lpc == 0
>
> Well, a negative value in use_lpc wouldn't initialize weight either.
> Are negative values intended?
Justin could probably comment but semantically it makes no sense.
also, this is supposed to be used internally so asserting should be ok.
anyhow, please wait for others to comment/approve/disapprove.
[...]
--
Regards,
Jai
More information about the ffmpeg-devel
mailing list