[MPlayer-dev-eng] [PATCH 1/5] Remove some useless static zero initializations.

Clément Bœsch ubitux at gmail.com
Sat Apr 30 23:35:01 CEST 2011


On Fri, Apr 22, 2011 at 11:47:00AM +0200, Reimar Döffinger wrote:
> On Sat, Apr 16, 2011 at 12:02:49AM +0200, Clément Bœsch wrote:
> > @@ -188,16 +188,16 @@ int fixed_vo=0;
> >  // benchmark:
> >  double video_time_usage=0;
> >  double vout_time_usage=0;
> > -static double audio_time_usage=0;
> > -static int total_time_usage_start=0;
> > -static int total_frame_cnt=0;
> > -static int drop_frame_cnt=0; // total number of dropped frames
> > +static double audio_time_usage;
> > +static int total_time_usage_start;
> > +static int total_frame_cnt;
> > +static int drop_frame_cnt; // total number of dropped frames
> >  int benchmark=0;
> 
> Outside of functions "static" or not makes no difference
> concerning this aspect, so you should also remove it
> from the others.

Mmmh indeed, that was stupid to ignore globals. I'll update the patch when
the git mirror will be back (is it already?). Should I commit it directly
if the make checkmd5sum is ok?

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20110430/36c39dbe/attachment.asc>


More information about the MPlayer-dev-eng mailing list