search

rust-crates / ergo_fs

ergonomic filesystem operations in rust
https://docs.rs/ergo_fs
Other
7 stars 0 forks source link

`PathArc`'s name exposes implementation detail #9

Open epage opened 6 years ago

epage commented 6 years ago

This was first discussed on reddit.

For "low level" APIs, having Arc in the name makes sense. If this is a higher level API meant to focus more on the Python type of users, we should abstract that implementation detail and leave it out of the name

vitiral commented 6 years ago

That would be fine. What name do you suggest?

PathArc was chosen because it mimicks PathBuf and gives the user immediate feedback that it is cheap to copy.

vitiral commented 6 years ago

Some possible names:

(I don't like any of these names personally, but thought it might get some ideas flowing)

epage commented 6 years ago

Why not just Path? Its what you get when you strip the suffix. I assume the rest are Arcs too and are missing the suffix.

vitiral commented 6 years ago

Mainly because I don't want to conflict with std::path::Path