[Piglit] [PATCH] The initial value for interval should be 0 to get the correct value.

Zhao, Jian J jian.j.zhao at intel.com
Thu Jul 14 19:56:28 PDT 2011


> -----Original Message-----
> From: Chad Versace [mailto:chad at chad-versace.us]
> Sent: Friday, July 15, 2011 12:58 AM
> To: Zhao, Jian J
> Cc: piglit at lists.freedesktop.org; Jesse Barnes
> Subject: Re: [Piglit] [PATCH] The initial value for interval should be 0 to get
> the correct value.
> 
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 07/13/2011 08:13 PM, Jian Zhao wrote:
> > There is some modification to the test before and now we need to adjust
> the
> >  intial value of the interval. And there is some comments updated.
> >
> > Signed-off-by: Jian Zhao <jian.j.zhao at intel.com>
> > CC: Jesse Barnes <jbarnes at virtuousgeek.org>
> > ---
> >  tests/glx/glx-swap-event.c |   10 +++++-----
> >  1 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/glx/glx-swap-event.c b/tests/glx/glx-swap-event.c
> > index c2ef60e..bc3f3f3 100644
> > --- a/tests/glx/glx-swap-event.c
> > +++ b/tests/glx/glx-swap-event.c
> > @@ -58,16 +58,16 @@ PFNGLXGETSWAPINTERVALMESAPROC
> pglXGetSwapIntervalMESA;
> >
> >  static GLboolean fullscreen = GL_FALSE;    /* Create a fullscreen
> window */
> >  static GLboolean verbose = GL_FALSE;    /* Disable verbose.  */
> > -static GLboolean Automatic = GL_FALSE;    /* Disable verbose.  */
> > -static GLboolean test_events = GL_FALSE;    /* Disable verbose.  */
> > -static GLboolean interval_diff = GL_FALSE;    /* Disable verbose.  */
> > -static GLboolean async = GL_FALSE;    /* Disable verbose.  */
> > +static GLboolean Automatic = GL_FALSE;    /*  Test automatically.  */
> > +static GLboolean test_events = GL_FALSE;    /* Test event's generation.
> */
> > +static GLboolean interval_diff = GL_FALSE;    /* Test interval can be set.
> */
> > +static GLboolean async = GL_FALSE;    /* Test asynchronize.  */
> >  int event_base, Glx_event, count=0, swap_count=0, event_count=0;
> >  static int Intel_swap_event=0;
> >  int  event_count_total=0, frames_total=0, message_count=0;
> >  static double time_call=0.0, time_fin=0.0, time_val=0.0;
> >  double swap_start[STACK_L],swap_returned[STACK_L];
> > -int interval=1;
> > +int interval=0;
> >  char * swap_event_type=NULL;
> >  /**
> >   * Determine whether or not a GLX extension is supported.
> 
> Please separate the comment modifications and the interval adjustment into
> separate commits.


OK. I have separated it into two small patches. Thanks. 

> 
> - --
> Chad Versace
> chad at chad-versace.us
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
> 
> iQIcBAEBAgAGBQJOHyA1AAoJEAIvNt057x8i1LQQAJ+C+DUlEoqHV5u6J5uDv2
> Qc
> ZztxXO2Qoitx6UEO7ex9JCleJHRiHMqvUFb0OHhljm69tFOLE8JpCJiuR6d246hc
> 1dXc+8qbhhbnjKvQnlm5947i47zvDIyqEyez+Qb2+UubFIXWdH6uwx/5apcPzAU
> 3
> BqeTK1PIMcEkG5UWJ5F5Ldx7qeVybyEqAd1ppHspbLEkQ9z60AkP4F0BWjTqv
> Xt2
> sztFjo6ItX33Dq74SZQVfqU4fGl8BzQ1Du7aRX2Bk8K0KAVOd1vYd8eMQSb5YI+
> 0
> RhJRoNyOacVfplKnHGg0tTIUkwal/KrmrkJGx4B8APcZ07cmT8a+dEXblaPNykIc
> fCLP6kCOnGwdV68ZS/RPYxwLTFcHR+HDGfw8jZgx0KgvTt4kt7tgheLiPzfW+2S
> A
> 4gj96JO56sFbULVvpzrxGWYV4jZ+QcOdd36HkQi/SJbWT1OTbAtCxRFp5UKf5K
> aG
> xOgF+0dK2OARVIgbsyIWlLlA72ThmVMNHFF2L6cILQFHegsZqel55mLZn7YiSO
> VF
> kQzM8/1pmqyq6RbOZJJM3aH/LZ2ctNYa9Zikct5kcTszX0QqfCOVTwkHYzeUzkJ
> Q
> NbOfnlRTH5lVE098p8MhBrCAXeeRFmYm4kdBUhs0Z7osTvrpPn2JBaIUCx3/Uz
> zW
> kjxXei5d8NFJkSC0JPTB
> =6RUf
> -----END PGP SIGNATURE-----


More information about the Piglit mailing list