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

Needed fix for benchmark #13

Open
DanielUranga opened this issue Jan 27, 2016 · 2 comments
Open

Needed fix for benchmark #13

DanielUranga opened this issue Jan 27, 2016 · 2 comments

Comments

@DanielUranga
Copy link

I've ported a benchmark that compares this version of Box2d to a native "linc" c++ one: https://github.com/DanielUranga/haxe_box2d_benchmarks I think this can be highly interesting to measure hxcpp performance vs native.

The problem is that the Haxe version crash due to this:
https://github.com/openfl/box2d/blob/master/box2D/dynamics/B2Body.hx#L77

Any chance we could fix it?

@jgranick
Copy link
Member

Can we compare to Box2DFlash and see if this is in the original code or not? Are you sure that's the cause of the crash?

@DanielUranga
Copy link
Author

This is the stacktrace:

Called from ? line 1
Called from MainBox2dHaxe.hx line 26
Called from box2D/dynamics/B2Body.hx line 169
Called from box2D/dynamics/B2Body.hx line 132
Called from box2D/dynamics/B2Fixture.hx line 320
Uncaught exception - Invalid field access : computeAABB

It calls m_shape.computeAABB(m_aabb, xf); on B2Fixture line 320, but m_shape is null.

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