forked/hyper
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #841 from hyperium/pr/833

Browse Source 
fix(headers): Remove raw part when getting mutable reference to typed header
This commit is contained in:
Sean McArthur
2016-06-20 16:22:20 -07:00
committed by GitHub
parent 6dab63fbac f38717e422
commit d02eb474e1
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/header/internals/item.rs
View File

@@ -78,6 +78,9 @@ impl Item {
Err(_) => () Err(_) => ()
} }
} }
if self.raw.is_some() && self.typed.get_mut(tid).is_some() {
self.raw = OptCell::new(None);
}
self.typed.get_mut(tid).map(|typed| unsafe { typed.downcast_mut_unchecked() }) self.typed.get_mut(tid).map(|typed| unsafe { typed.downcast_mut_unchecked() })
} }
} }

1
src/header/mod.rs
View File

@@ -670,6 +670,7 @@ mod tests {
fn test_get_mutable() { fn test_get_mutable() {
let mut headers = Headers::from_raw(&raw!(b"Content-Length: 10")).unwrap(); let mut headers = Headers::from_raw(&raw!(b"Content-Length: 10")).unwrap();
*headers.get_mut::<ContentLength>().unwrap() = ContentLength(20); *headers.get_mut::<ContentLength>().unwrap() = ContentLength(20);
assert_eq!(headers.get_raw("content-length").unwrap(), &[b"20".to_vec()][..]);
assert_eq!(*headers.get::<ContentLength>().unwrap(), ContentLength(20)); assert_eq!(*headers.get::<ContentLength>().unwrap(), ContentLength(20));
} }