On 13/12/13 16:12, Vittorio Giovara wrote:
> On Fri, Dec 13, 2013 at 12:28 AM, Guillaume Martres <smar...@ubuntu.com>
> wrote:
>> From: Michael Niedermayer <michae...@gmx.at>
>>
>> ---
>> libavcodec/hevc_sei.c | 5 ++++-
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavcodec/hevc_sei.c b/libavcodec/hevc_sei.c
>> index eed7b33..88b4744 100644
>> --- a/libavcodec/hevc_sei.c
>> +++ b/libavcodec/hevc_sei.c
>> @@ -49,7 +49,10 @@ static void
>> decode_nal_sei_decoded_picture_hash(HEVCContext *s)
>> static void decode_nal_sei_frame_packing_arrangement(HEVCContext *s)
>> {
>> GetBitContext *gb = &s->HEVClc.gb;
>> - int cancel, type, quincunx, content;
>> + int cancel;
>> + int quincunx = 0;
>> + int content = -1;
>> + int type = -1;
>
> Actually these variables are not used at all when unintialized, as
> they are all guarded by proper "frame_packing_present_flag" checks.
> What this patch fixes is just a few warnings from clang, I'll amend
> the commit message to reflect this.
wait, why clang is issuing warnings? the static analyzer should tell us
more.
lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
