thomthom / quadface-tools

Suite of tools for manipulating quad faces in SketchUp.

https://extensions.sketchup.com/en/content/quadface-tools

MIT License

18 stars 1 forks source link

Redesign EntitiesProvider #59

Open thomthom opened 13 years ago

thomthom commented 13 years ago

Original report by me.

Make EntitiesProvider keep a cache of faces => QuadFace for reuse of QuadFace objects. While keeping a separate record of what EntitiesProvider contains.

This means that EntitiesProvider.include? Should not check the face => QuadFace.

Then get and [] can be aliases of each others.

thomthom commented 12 years ago

Possible bug with #add and/or #get.

get doesn't seem to reuse previously processed quads - returns a new QuadFace for each given Sketchup::Face (maybe QuadFace?)