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

Fixing the ODE Library and adding support for DrawStuff Library #251

Merged
merged 2 commits into from
Sep 18, 2013

Conversation

Teslos
Copy link
Contributor

@Teslos Teslos commented Sep 18, 2013

Hi,
There was similar problem in ODE library like in RegexMatch case .
For ODEJoint the problem was in code:

return IoODEJoint_newProtoCommon(state, IoODEBall_proto, jointGroup);

this code passes IoODEBall_proto as IoStateProtoFunc which is then passed to:

IoODEJoint *proto = IoState_protoWithId_(state, func);

as function pointer instead as protoId which is char *.
Changing previous line to:

IoODEJoint *proto = IoState_protoWithId_(state, protoWithId);

It was necessary to change functions calls to all possible Joints object IODEBall, IODEHinge etc..

The second change is addition of the DrawStuff library for drawing the ODE objects, it provides visualization
of the open dynamics engine and requires much less code then OpenGL and GLUT examples.
The screenshot shows the DrawStuff example similar to dstest example in ode-0.12 distribution:
frame0001
All examples with DrawStuff support are in addons/ODE/samples/drawstuff directory.

The samples are tested on my mac: 12.5.0 Darwin Kernel Version 12.5.0: Mon Jul 29 16:33:49 PDT 2013; root:xnu-2050.48.11~1/RELEASE_X86_64 x86_64

Please can you test the examples on your configuration too. It is necessary to have drawstuff library in
/usr/local/lib/libdrawstuff.la and libdrawstuff.a
textures are saved for now in /usr/local/include/drawstuff.

Best regards,
∃t

stevedekorte added a commit that referenced this pull request Sep 18, 2013
Fixing the ODE Library and adding support for DrawStuff Library
@stevedekorte stevedekorte merged commit 5d35419 into IoLanguage:master Sep 18, 2013
@stevedekorte
Copy link
Member

Thanks Teslos :)

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.

2 participants