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

timer.addPeriodic doesn't return the good function #8

Closed
plule opened this issue Sep 25, 2012 · 1 comment
Closed

timer.addPeriodic doesn't return the good function #8

plule opened this issue Sep 25, 2012 · 1 comment

Comments

@plule
Copy link

plule commented Sep 25, 2012

I'm not totally sure about this, but addPeriodic seems to return a function which will only last during the first cycle. After that, timer.add() is called again and the return value is not valid anymore. Thus timer.cancel doesn't work with periodics.

I couldn't work on a real fix (I encounter it during miniLD, so not much time), but I have an ugly workaround (separating periodics and non periodics) if you are interested.

@plule
Copy link
Author

plule commented Sep 25, 2012

Matter of fact, I can't reproduce it on a simple example and it seems that this bug doesn't exists. I must have messed up something. Sorry about that :)

@plule plule closed this as completed Sep 25, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant