search

chipsenkbeil / typed-path

Provides typed variants of Path and PathBuf for Unix and Windows
38 stars 6 forks source link

Utf8UnixPath as_str() and to_string() #7

Closed hjmallon closed 1 year ago

hjmallon commented 1 year ago

This could just be a bug on my side, but I found it confusing that Utf8UnixPath::new("my/path").to_string() has " around the path, when none of the other methods (as_str(), into_string() etc) do.

std::path::Path doesn't seem to have the same behaviour (it seems to never add "s)

chipsenkbeil commented 1 year ago

Sounds like a bug to me! I'll look into why and get a fix out.

chipsenkbeil commented 1 year ago

@hjmallon can you test this with v0.3.2? Should fix your issue.

hjmallon commented 1 year ago

Thanks again! 0.3.2 fixes the issue for me. Also thanks for such speedy responses.

chipsenkbeil commented 1 year ago

Thanks again! 0.3.2 fixes the issue for me. Also thanks for such speedy responses.

@hjmallon thanks for finding and reporting these issues with a clear problem and expectation :) I wrote this library to solve a need for myself, and it's good to see some of the rough edges getting worked out!