improved collision callback

[chandlerprall]

improved collision callback

[chandlerprall]

Absolutely - what do you think about leaving out the clone() call altogether? If you want to store the vectors it's left to the dev to explicitly clone?

In fact I've thought about doing this for most of the vectors passed to the callbacks, but I'm afraid that kind of thing is hard to document and ensure everyone understands that those are flyweights.

I suppose some simple testing would show if making such a change yields any perceivable increase in performance.

I only added in clone() because _temp2 overwrites _temp1, but yeah, as long as the vectors in the callback dont get overwritten afterwards, it'll work!

[chandlerprall]

Should be good now :) It is using flyweights so make sure to clone() if you end up storing the values.

sweet, thanks!