Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix sys.time overflow #254

Closed
GoogleCodeExporter opened this issue Oct 6, 2015 · 3 comments
Closed

Fix sys.time overflow #254

GoogleCodeExporter opened this issue Oct 6, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

Report from Marcus Priesch via email on 2013/11/01::

Hello Dean,

today i did some testing ... and sys.time () overflow every 49 days caused me
some headaches ... ,)

with your implementation sys.time yould have thrown a type error in the
latter 49/2 days ;) - which nevertheless would have restarted the application
in my case ... also the scheduler would have stopped after an oferrun ...

this led me to the attached patch ...

sys.time () now only returns msecs until sys.maxint is reached, when
overflowing, sys.wait () now also handles this correctly ...

feel free to adapt ;)

regards,
marcus.

Original issue reported on code.google.com by dwhall...@gmail.com on 5 Nov 2013 at 4:10

Attachments:

@GoogleCodeExporter
Copy link
Author

Resolved by r713d9237869c

Original comment by dwhall...@gmail.com on 5 Nov 2013 at 4:42

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

I added an else clause to Marcus' path and that was not correct.  I need to 
revert to exactly match the patch.

Original comment by dwhall...@gmail.com on 6 Nov 2013 at 3:20

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision 5dc724e5104b.

Original comment by dwhall...@gmail.com on 6 Nov 2013 at 3:22

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant