refactored Pop() function on threadUnsafeSet

[notEpsilon]

old

func (s *threadUnsafeSet[T]) Pop() (v T, ok bool) {
    for item := range *s {
        delete(*s, item)
        return item, true
    }
    return
}

in my opinion, the last return badly impacts readability.

refactored

func (s *threadUnsafeSet[T]) Pop() (T, bool) {
    for item := range *s {
        delete(*s, item)
        return item, true
    }
    return *new(T), false
}

• improved readability.
• no usage of named returns since they weren't actually used, they existed as a hack to return default values.

test results

$ go test ./... -cover -race
ok      github.com/deckarep/golang-set/v2       79.012s coverage: 94.8% of statements