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

A few changes #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

A few changes #2

wants to merge 4 commits into from

Conversation

TimMensch
Copy link

I needed the ability to use the # operator on arrays passed down from Java. It's a pretty simple change, and I've tested it and it works.

I also needed LuaObject.finalize() to not block on synchronized(L).

As a bonus, I hate seeing warnings, so I found the GCC pragma that asks it to ignore the warnings in question, as well as cleaning up the warnings in LuaObject.

I also have extensions allowing LuaJava to access LuaJIT's FFI. Let me know if you're interested in adding that; FFI is awesome, and works on Android to pull in and link with .so files (as well as standard C library functions, of course -- I got rid of several calls into Java entirely by using FFI calls into C libraries, which is just all kinds of faster).

--Tim

Tim Mensch added 4 commits February 8, 2013 12:16
…creating a new object.

This way a long-running Lua thread won't cause the garbage collector to time out blocking on synchronized(L).
@vadi2
Copy link

vadi2 commented Mar 19, 2013

Finalization of Lua objects is something that's been hitting me on long network requests, I'll be testing this out myself as well - thanks!

@CameraCornet
Copy link

what's the status on this being accepted?

@stevedonovan
Copy link
Owner

Soon, I promise you.

On Sun, Nov 30, 2014 at 12:14 AM, CameraCornet notifications@github.com
wrote:

what's the status on this being accepted?


Reply to this email directly or view it on GitHub
#2 (comment).

@TimMensch
Copy link
Author

TimMensch commented Sep 20, 2018

I'm not using AndroLua any more, so it's no skin off of my nose, but I was amused when I looked at my open pull requests to see the "Soon, I promise you." above.

Oops. ¯\(ツ)

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

Successfully merging this pull request may close these issues.

None yet

4 participants