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

Too many callbacks #3

Open
LPGhatguy opened this Issue Oct 14, 2015 · 6 comments

Comments

Projects
None yet
3 participants
@LPGhatguy
Owner

LPGhatguy commented Oct 14, 2015

I got a request about this via email, stating:

"I'm using luajit-request in order to get a lot of pages, so eventually it ends up failing because of too many callbacks (on luajit-request/init.lua:148).
Would you know what I could do to avoid doing so many callbakcs, or to free them when I done using them?"

@LPGhatguy LPGhatguy added the bug label Oct 14, 2015

@shakesoda

This comment has been minimized.

Show comment
Hide comment
@shakesoda

shakesoda Oct 14, 2015

yeah, this is one of those horribly sneaky FFI gotchas. they can be freed, however, according to the docs

shakesoda commented Oct 14, 2015

yeah, this is one of those horribly sneaky FFI gotchas. they can be freed, however, according to the docs

@LPGhatguy

This comment has been minimized.

Show comment
Hide comment
@LPGhatguy

LPGhatguy Dec 17, 2015

Owner

If anyone wants to take a look at this, especially someone who uses the library, I'd really appreciate it.

Owner

LPGhatguy commented Dec 17, 2015

If anyone wants to take a look at this, especially someone who uses the library, I'd really appreciate it.

@LPGhatguy

This comment has been minimized.

Show comment
Hide comment
@LPGhatguy

LPGhatguy Dec 19, 2015

Owner

I believe I'll have this fixed in the next push up!

Owner

LPGhatguy commented Dec 19, 2015

I believe I'll have this fixed in the next push up!

@LPGhatguy LPGhatguy added fixed and removed help wanted labels Dec 19, 2015

@LPGhatguy

This comment has been minimized.

Show comment
Hide comment
@LPGhatguy

LPGhatguy Dec 19, 2015

Owner

Done in 2.3.0!

Owner

LPGhatguy commented Dec 19, 2015

Done in 2.3.0!

@LPGhatguy LPGhatguy closed this Dec 19, 2015

@Fallstar

This comment has been minimized.

Show comment
Hide comment
@Fallstar

Fallstar Dec 20, 2015

Hi, just tried it, still does too many callbacks. :/
I'll try to make a test suite when I'm done with my other projects.

Fallstar commented Dec 20, 2015

Hi, just tried it, still does too many callbacks. :/
I'll try to make a test suite when I'm done with my other projects.

@LPGhatguy LPGhatguy reopened this Dec 21, 2015

@LPGhatguy LPGhatguy removed the fixed label Dec 21, 2015

@LPGhatguy

This comment has been minimized.

Show comment
Hide comment
@LPGhatguy

LPGhatguy Dec 21, 2015

Owner

That'd be much appreciated!

Owner

LPGhatguy commented Dec 21, 2015

That'd be much appreciated!

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