Skip to content

Simplify Vertices and Edges collections, remove unneeded factory methods and favor collections over simple arrays#188

Merged
clue merged 3 commits intographp:masterfrom
clue-labs:simplify-sets
Jan 20, 2020
Merged

Simplify Vertices and Edges collections, remove unneeded factory methods and favor collections over simple arrays#188
clue merged 3 commits intographp:masterfrom
clue-labs:simplify-sets

Conversation

@clue
Copy link
Member

@clue clue commented Jan 20, 2020

// old
$vertices = Vertices::factory($arrayOfVertices);
$vertices->getVerticesIntersection($arrayOfVertices);

// new (already supported before)
$vertices = new Vertices($arrayOfVertices);
$vertices->getVerticesIntersection(new Vertices($arrayOfVertices));

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant