Deuxfleurs/garage
44
56
Fork 74
Code Issues 100 Pull requests 14 Releases 30 Wiki Activity

Support website publishing #7

Merged
lx merged 61 commits from feature/website into master 2021-01-15 16:49:51 +00:00
Conversation 3 Commits 61 Files changed 17 +1 -3
1 changed files with 1 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit f8a40e8c4f - Show all commits

4
src/web/web_server.rs
View file

@ -176,9 +176,7 @@ fn path_to_key<'a>(path: &'a str, index: &str) -> Result<Cow<'a, str>, Error> {
} }
match path_utf8.chars().last() { match path_utf8.chars().last() {
quentin marked this conversation as resolved Outdated
lx commented 2021-01-13 21:18:51 +00:00
Outdated
Review
Copy link

this case is unreachable!() thanks to the previous if

this case is `unreachable!()` thanks to the previous if
None => Err(Error::BadRequest(format!( None => unreachable!(),
"Path must have at least a character"
))),
Some('/') => { Some('/') => {
let mut key = String::with_capacity(path_utf8.len() + index.len()); let mut key = String::with_capacity(path_utf8.len() + index.len());
key.push_str(&path_utf8[1..]); key.push_str(&path_utf8[1..]);