schteppe / p2.js

JavaScript 2D physics library
Other
2.64k stars 330 forks source link

Circular dependency between Ray and RaycastResult #249

Closed AndrewRayCode closed 5 years ago

AndrewRayCode commented 8 years ago

Ray.js requires RaycastResult, even though it's not used as far as I can tell, and RaycastResult.js requires Ray.

I'm working on a tool to detect cyclical requires in nodejs projects and it detected this one. I think the require can just be removed from Ray.js?

schteppe commented 8 years ago

I think you accidentally mistook the unit test files (test/) for the source files (src/)?