[FFmpeg-devel] [PATCH] fix uninitialized variable warning in seek_test.c

Michael Niedermayer michaelni
Thu Mar 26 15:11:09 CET 2009


On Thu, Mar 26, 2009 at 02:21:17PM +0100, Benoit Fouet wrote:
> On 03/26/2009 02:02 PM, Diego Biurrun wrote:
> > On Thu, Mar 26, 2009 at 01:56:22PM +0100, Benoit Fouet wrote:
> >   
> >> attached is a patch to $subj
> >>
> >> --- tests/seek_test.c	(revision 18194)
> >> +++ tests/seek_test.c	(working copy)
> >> @@ -70,7 +70,7 @@ int main(int argc, char **argv)
> >>  
> >>      for(i=0; ; i++){
> >>          AVPacket pkt;
> >> -        AVStream *st;
> >> +        AVStream *av_uninit(st);
> >>     
> >
> > The #include is missing I think.
> >
> >   
> 
> indeed, fixed
> 
> > Does performance matter in any way here?  Otherwise it might be simpler
> > to initialize the variable.
> >
> >   
> 
> I don't care, I just wanted to fix the false warning :)

patch ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090326/d9cff18e/attachment.pgp>



More information about the ffmpeg-devel mailing list