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

Already on GitHub? Sign in to your account

Fixing ttl calculation #9

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

Maybe it depends on your redis version, but when I compare the ttl calculation of redis with the one in MockRedis, it always differ by 1.

I use Redis version 2.2.4

Feel free to merge

@ghost

ghost commented Mar 5, 2012

Thanks for the patch. I've amended it to make the minimal changes to the spec files rather than rewriting all the examples, and I also replaced your fix (adding a digit, which doesn't feel right) with something that seems a little more transparent and robust (using integers when calculating the difference between times in the #ttl method).

@ghost ghost closed this Mar 5, 2012

@sds sds added the bug label Apr 8, 2014

This issue was closed.

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