net/url: why not return dst.String() directly?

[rising622]

go/src/net/url/url.go

Line 1048 in 304f769

return "/" + strings.TrimPrefix(dst.String(), "/")

why not return dst.String() directly

[mvdan]

Most likely because you don't know if dst has a slash as a prefix?

The code doesn't seem terribly efficient in the case where the slash already exists, but I don't know if that actually matters.

[ianlancetaylor]

It's for historical reasons. See https://golang.org/cl/264817.

But we can clean it up. Sent https://golang.org/cl/282673.

[gopherbot]

Change https://golang.org/cl/282673 mentions this issue: net/url: avoid string concatenation in resolvePath