tzaeschke / tinspin-indexes

Spatial index library with R*Tree, STR-Tree, Quadtree, CritBit, KD-Tree, CoverTree and PH-Tree
http://www.tinspin.org
Apache License 2.0
111 stars 24 forks source link

IndexConfig's factory 'create' method's supposed to be static? #35

Closed LerinRuss closed 11 months ago

LerinRuss commented 1 year ago

Hi, good evening. I'm revisiting issue #33 and I noticed that the create factory method of org.tinspin.index.IndexConfig is no static. Could you please clarify if this change is intentional?

tzaeschke commented 1 year ago

No, this is not intentional. It it actually quite embarrassing, it just goes to show that even seemingly simple changes deserve a unit test.

I already made a new release but it may take a day or so until it is indexed in the major maven search sites.

Thanks a lot for reporting!