vers icon indicating copy to clipboard operation
vers copied to clipboard
verified publisher icon Cydhra

Minor inconsistencies in API naming

Open faassen opened this issue 11 months ago • 3 comments

While you create an RsVec from a bit vector using from_bit_vec, the BpTree is created using from_bit_vector. Perhaps from_bit_vector can be deprecated in favor of from_bit_vec?

faassen avatar May 21 '25 13:05 faassen

I also noticed it's SparseRSVec and not SparseRsVec, while it's RsVec.

faassen avatar May 21 '25 13:05 faassen

Relates to #9

Cydhra avatar May 21 '25 14:05 Cydhra

The deprecation for the top one could be done. Whether a second deprecation for correcting the capitalization is useful, I don't know. When I push a 2.0 update, downstream crates will need major updates anyway to migrate, so I think the deprecations aren't very useful in both cases.

I noticed the wrong capitalization about 5 minutes after I released the 1.6 update. I was very annoyed :P

Cydhra avatar May 21 '25 20:05 Cydhra