On 2/23/2011 10:01 AM, James Westby wrote:
> Review: Approve
> 94 + cmd = 'import time; while True: time.sleep(0.001)'
>
> Should this have a timeout?
>
> Other than that this looks good.
>
> Thanks,
>
> James
>
It intentionally doesn't, because it is testing the kill code. But you
have a point, setting it to:
for i in xrange(10000): time.sleep(0.001)
would give us 10s of a process running, and not leave something forever
if the test fails for some reason.
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 2/23/2011 10:01 AM, James Westby wrote:
> Review: Approve
> 94 + cmd = 'import time; while True: time.sleep(0.001)'
>
> Should this have a timeout?
>
> Other than that this looks good.
>
> Thanks,
>
> James
>
It intentionally doesn't, because it is testing the kill code. But you
have a point, setting it to:
for i in xrange(10000): time.sleep(0.001)
would give us 10s of a process running, and not leave something forever
if the test fails for some reason.
John
=:->
-----BEGIN PGP SIGNATURE----- enigmail. mozdev. org/
lPsMACgkQJdeBCY SNAAN9ewCfZ5GCU l4wYCdZuOvXbHb1 47+7 mtIPzFF/ z4LQNo2kW
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAk1
BckAoLeQrQTvbLH
=a/qV
-----END PGP SIGNATURE-----