simple: check uneeded []byte or []string conversion for len builtin

[martisch]

Converting a string or []byte to the other type to compute the length is not needed.

Example:
https://github.com/golang/net/blob/0ba52f642ac2f9371a88bfdde41f4b4e195a37c0/dns/dnsmessage/message.go#L1848

[Antonboom]

Looks like a typo in the title.
Not []byte or []string, but []byte or string.

[Antonboom]

+ json.RawMessage conversion

[ccoVeille]

Indeed,

len([]byte(...: 6.5k

len(string(: 5.4k (but they are struct with stringer among them)

len(json.RawMessage(...: 4

[ccoVeille]

@Antonboom

Does the code you tried when playing with testifylint around len(string( and len([]byte( could be used to help to provide the linter needed here

[Antonboom]

could be used to help to provide the linter needed here

I don't mind if it makes sense.

[ccoVeille]

My question was somehow to ask you if you plan to open a PR for fixing this issue

[Antonboom]

you plan to open a PR for fixing this issue

I don't plan to, it's out of my focus at the moment. My apologies 🙏