A poll: whether almost all function/method declarations should come with named return values unconditionally

[zerkms]

For code like

func (u *UUID) decodeHashLike(t []byte) (err error) {
    src := t[:]
    dst := u[:]

    if _, err = hex.Decode(dst, src); err != nil {
        return err
    }
    return
}

or

func (u UUID) MarshalBinary() (data []byte, err error) {
    data = u.Bytes()
    return
}

named return values to me don't add any value but mental overhead.

I suggest to vote whether we keep it as-is (then the "rule" should be standardised - so that it was applied consistently, now not all functions have it) or remove, unless it really is one of those rare cases when it makes something "more readable" (and yes, I understand it's subjective).

🎉 - to keep and standardise
❤️ - to remove