Fix: BBTree.log() #19

Merged
merged 5 commits into from Feb 1, 2013

Conversation

Projects
None yet
2 participants
@robwalch
Contributor

robwalch commented Jan 27, 2013

I was wondering what 'nodeRender' does, and found it was broken - probably not updated in a while. Here's the fix.

@robwalch

This comment has been minimized.

Show comment Hide comment
@robwalch

robwalch Jan 29, 2013

Contributor

Found some bad refs down under body.sleep(). Fixed even though it seems like a bad idea to call this. CP should be in charge or activating and deactivating bodies. It'll definitely cause issues if you do it to a body with contacts.

Contributor

robwalch commented Jan 29, 2013

Found some bad refs down under body.sleep(). Fixed even though it seems like a bad idea to call this. CP should be in charge or activating and deactivating bodies. It'll definitely cause issues if you do it to a body with contacts.

@josephg

This comment has been minimized.

Show comment Hide comment
@josephg

josephg Jan 29, 2013

Owner

Good catch - cheers

Owner

josephg commented Jan 29, 2013

Good catch - cheers

@josephg

This comment has been minimized.

Show comment Hide comment
@josephg

josephg Jan 29, 2013

Owner

Can't automerge...

Owner

josephg commented Jan 29, 2013

Can't automerge...

@robwalch

This comment has been minimized.

Show comment Hide comment
@robwalch

robwalch Jan 29, 2013

Contributor

Sorry, I was missing some updates. I rebased upstream and merged.

Contributor

robwalch commented Jan 29, 2013

Sorry, I was missing some updates. I rebased upstream and merged.

josephg added a commit that referenced this pull request Feb 1, 2013

@josephg josephg merged commit faa3f91 into josephg:master Feb 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment