[Bf-codereview] Bullet Integration (issue 7092051)
ideasman42 at gmail.com
ideasman42 at gmail.com
Tue Jan 15 17:08:58 CET 2013
Some things I should have mentioned last review...
- The way groups is used I find a bit strange, instead it could flag
objects and store in an array? - it works as is and may be annoying to
change right now but noting here.
Maybe other devs can explain why doing it this way is good.
- Think it would be good to have a check on the scene if physics should
run something like;
'bool BKE_scene_rigidbody_check(Scene *scene)',
The check only needs to see if the scene has a physics pointer and its
un-muted.
Then any place physics is done through the code this check can be
called.
Currently each function has to check in its body which I find less easy
to follow.
https://codereview.appspot.com/7092051/
More information about the Bf-codereview
mailing list