Steven Bosnick

Document and test the use of unsafe

5

A recent Medium [article](https://medium.com/@shnatsel/smoke-testing-rust-http-clients-b8f2ee5db4e6) criticized 3 of the 7 rust http clients it reviewed because of their dependency on this crate and because of this crate's use of `unsafe`. The...

Audit the use of unsafe in header/value.rs

2

Reorganized the `Debug` impl for `HeaderValue` to make the invariants that its use of `unsafe` rely upon for soundness more obvious. This also adds a few more test cases for...

Audit the use of unsafe in uri/path.rs

4

Reorganize the one function with a use of unsafe (from_shared()) to highlight that it does a scan of each byte that it eventually passes to ByteStr::from_utf8_unchecked(). This makes it apparent...

Added tests for attempts to parse various types of invalid `Uri`'s including ones with invalid UTF-8 bytes in them. Added a test for parsing `&[u8]` as a Uri where it...

Consider adding an executor for vimux

1

It may be useful to add a third executor for the `RustRunnables` support that makes use of the [preservim/vimux](https://github.com/preservim/vimux) plugin for vim/tmux integration. I have a version of this working...

The example in README.md lists several packages as dependencies of kcov, but does not list "libiberty-dev". The CMake.txt script used to configure the kcov build looks for "libiberty" as a...