[PATCH x11perf] remove srand(1) call as nothing calls any rand like functions in x11perf

Thomas Klausner wiz at NetBSD.org
Mon Jan 12 02:44:06 PST 2015


On Mon, Jan 12, 2015 at 11:28:32AM +0100, Matthieu Herrb wrote:
> On Sun, Jan 11, 2015 at 11:40:32AM +0100, Thomas Klausner wrote:
> > LGTM
> 
> Hi Thomas,
> 
> Can I interpret that as a Reviewed-by: header? 

Yes, sorry for not being more explicit.
 Thomas

> > 
> > On Sat, Jan 10, 2015 at 10:51:12AM +0100, Matthieu Herrb wrote:
> > > Signed-off-by: Matthieu Herrb <matthieu at herrb.eu>
> > > ---
> > >  x11perf.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > > 
> > > diff --git x11perf.c x11perf.c
> > > index 6325ad6..3872bcb 100644
> > > --- x11perf.c
> > > +++ x11perf.c
> > > @@ -845,7 +845,6 @@ ProcessTest(XParms xp, Test *test, int func, unsigned long pm, char *label)
> > >      DisplayStatus(xp->d, "Calibrating", label, 0);
> > >      reps = CalibrateTest(xp, test, seconds, &time);
> > >      if (reps != 0) {
> > > -	srand(1);	/* allow reproducible results */
> > >  	XDestroySubwindows(xp->d, xp->w);
> > >  	XClearWindow(xp->d, xp->w);
> > >  	reps = (*test->init) (xp, &test->parms, reps);
> > > -- 
> > > 1.9.3
> > > 
> > > _______________________________________________
> > > xorg-devel at lists.x.org: X.Org development
> > > Archives: http://lists.x.org/archives/xorg-devel
> > > Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list