A BTreeSet would also give you order. That might be preferable if there are a lot of duplicates, and you don't want the overhead of storing them before vec sort+dedup.
Right; it's less efficient due to the extra allocation & copy. That may be an acceptable price to pay for the simpler code, especially in a prototyping stage.