search

Ralith / sieve-tree

6 stars 1 forks source link

Order of `(id, bounds)` pairs #5

Closed grovesNL closed 6 days ago

grovesNL commented 1 week ago

This is minor nitpicky thing but intersections provides (id, bounds) but remove accepts bounds, id.

These should probably have the same order for consistency, because sometimes you'll want to take intersection candidates and pass them to remove later.

Ralith commented 6 days ago

Standardized on id, bounds, [value].