I made two patches to fix some things in clutter-box2d: a patch to fix
the frequency and damping arguments of the
clutter_box2d_add_distance_joint method, and a patch that fixes the
method clutter_box2d_actor_get_property in clutter-box2d-actor.cpp/h so
that the property PROP_LINEAR_VELOCITY is gotten (previously, this
function was setting it, not getting it).

I submitted both the patches on the bugzilla site.  If you could, please
take a look at them.

A few notes... 
My first patch fixes the frequency and damping ratio when creating a
distance joint in the function clutter_box2d_add_distance_joint in the
file clutter-box2d-joint.cpp. Previously, the function was not using the
inputted arguments frequency and damping_ratio when creating a box2d
distance joint. Instead, it was setting the b2DistanceJointDef's
frequencyHz and dampingRatio properties to 0.  

I noticed that many of clutter-box2d-joint's input arguments required
scaling (multiplied by the SCALE_FACTOR factor of 0.05).  I couldn't
find adequate documentation on what reasonable values for frequency and
damping ratio should be from either clutter-box2d or Box2D.  I did look
up some physics documents explaining what these values mean in the real
world.  I also tested many values for each programmatically.  To the
best of my knowledge, these arguments do not need to be multiplied by
SCALE_FACTOR, but should be left as-is.  Please let me know if they
should be scaled, and I will fix this.

My second patch fixes the method clutter_box2d_actor_get_property in the
file clutter-box2d-actor.cpp/h so that you can get the property
PROP_LINEAR_VELOCITY.  

Let me know if you want me to submit the patches via the mailing list,
or if the bugzilla is sufficient.

Thanks!
Lilli 
--
To unsubscribe send a mail to [EMAIL PROTECTED]

Reply via email to