On Fri, Dec 13, 2013 at 12:28 AM, Guillaume Martres <[email protected]> wrote: > From: Michael Niedermayer <[email protected]> > > --- > 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. Vittorio > > get_ue_golomb(gb); // frame_packing_arrangement_id > cancel = get_bits1(gb); // frame_packing_cancel_flag > -- > 1.8.3.2 > _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
