search

alexbol99 / flatten-js

Javascript library for 2d geometry
MIT License
546 stars 56 forks source link

`CircularLinkedList` not exported #90

Closed MaikuMori closed 3 years ago

MaikuMori commented 3 years ago

These utility classes are useful and should be exported:

alexbol99 commented 3 years ago

Hi @MaikuMori ,

These two structures intended for internal usage, they represent list of edges. Implementation is not generic, you cannot use them as a list of arbitrary items. I am not sure that expose them is a good idea. There are a lot of packages implementing linked list in npm. Why do we need another one?

Thanks, Alex

MaikuMori commented 3 years ago

Fair enough, I was just playing around with the library, and they would have been useful. I ended up just using an array and simulating a linked list.

Feel free to close the issue.