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

Compilation fails on neko #5

Closed
drkibitz opened this issue Oct 12, 2013 · 4 comments
Closed

Compilation fails on neko #5

drkibitz opened this issue Oct 12, 2013 · 4 comments

Comments

@drkibitz
Copy link

Using the openfl-sample SimpleBox2D openfl/openfl-samples#8.
This is the result from test with Haxe 3.0.1 and Neko 2.0.0 on a Mac 64.

$ openfl test neko
Called from box2D/common/B2Settings.hx line 72
Uncaught exception - Invalid operation (*)
@drkibitz
Copy link
Author

@jgranick Ok, I did a little digging, but first I have a question. How are you maintaining this library? Is it a manual port or is it transpiled? I'm asking because making specific changes will make it much harder to maintain if it is the latter :/

So my findings are, that first compilation error is just an order of definitions issue. Line 72 just doesn't know about the constant it is using yet.

After fixing this, there are numerous runtime errors, that I have seemed to boil down to one simple thing. This library is expecting null to be coerced to 0 in quite a few operations involving mainly bitmasks and increments from what I can see.

I just kept stomping them and making sure a 0 was defined where these operations failed with null.

With that the openfl sample works on neko.

Your thoughts?

@drkibitz
Copy link
Author

Since I'm new to the party, I'm not sure if this is a recent change in Neko, or in Haxe targeting Neko. But it does seem like a weird change :/

@jgranick
Copy link
Member

This was a manual conversion, so all (or any) Neko fixes would be greatly appreciated!

@jgranick
Copy link
Member

Box2D works on Neko in my tests, but if you have any more problems, please report them. Thank you! 😉

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

2 participants