fredbi / uri / 6245655794

89%

push

github

perf: implemented profile-driven optimizations (#33)

* Added a test to capture profile (there is a build tag guard to avoid pulling dependencies)
* Enriched benchmarks to capture various payloads and compare with
`net/url.Parse()`
* Reduced memory allocations during parsing and validations (see below)

With these optimizations, `uri.Parse()` is only about 30% slower than `net/url.Parse()` (with go1.20).

* [x] reduced extraneous allocations spotted [in this WIP PR](https://github.com/fredbi/uri/pull/33)
* [x] gave up the convenience of using `strings.Split()` to get a zero-allocation iterator over segments
* [x] reduced the need to systematically call `url.PathUnescape()`: call
only when a % is detected
* [x] moved the internal pointer to `authorityInfo` to a plain struct: removes 1 extraneous allocation, simpler code

Signed-off-by: Frédéric BIDON <fredbi@yahoo.com>

101 of 101 new or added lines in 1 file covered. (100.0%)

704 of 716 relevant lines covered (98.32%)

791.57 hits per line