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

[FR] Please include AllHitsRayResultCallback in haxelib #26

Closed
trsh opened this issue Feb 7, 2019 · 8 comments
Closed

[FR] Please include AllHitsRayResultCallback in haxelib #26

trsh opened this issue Feb 7, 2019 · 8 comments

Comments

@trsh
Copy link

trsh commented Feb 7, 2019

We have encountered some situations where it becomes very handy for armory project.

@zicklag
Copy link

zicklag commented Feb 8, 2019

I'm working on updating the way that the Bullet classes are bound to Haxe so I'll look into exposing that when I get it working. We should just have to add the class to the WebIDL file when I finish with #25.

@trsh
Copy link
Author

trsh commented Feb 9, 2019

@zicklag will it be cross Platform, also HTML-JS builds?

@zicklag
Copy link

zicklag commented Feb 9, 2019

Yep, it should work for HL/C, Krom, and HTML5 builds.

@trsh
Copy link
Author

trsh commented Feb 10, 2019

I see there is callback function addSingleResult, that fixates hit object. If we can expose and override that, we can create our own callbacks. Like secondHitRayResultCallback :)

@zicklag
Copy link

zicklag commented Feb 10, 2019

I wanted to do that, too. When first I tried Krom would crash, so I'm not 100% sure whether or not it is possible to extend a C++ ( compiled to JavaScript ) class in the plain JavaScript Haxe generates. I haven't done a lot of testing, so I'll have to try it again later.

I've almost got the auto-binder finished, though, so that could help make it easier to add custom callbacks by writing C++ if it was necessary.

@trsh
Copy link
Author

trsh commented Feb 17, 2019

@zicklag how far is this?

@zicklag
Copy link

zicklag commented Feb 17, 2019

I've got open PRs that are awaiting review by Lubos and Robert: #25 (comment). As soon as that is finished ( or maybe before ) I'll add the bindings for AllHitsRayResultCallback. It is a big change so it will take longer to review and test so it might be just a little bit.

@luboslenco
Copy link
Member

Link with #25.

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

3 participants