Why is ConcurrentMap sometimes passed by value and sometimes by reference?

[taybin]

Why is ConcurrentMap sometimes passed by value and sometimes by reference?

// Returns shard under given key
func (m ConcurrentMap) GetShard(key <KEY>) *ConcurrentMapShared {
    hasher := fnv.New32()
    hasher.Write([]byte(key))
    return m[int(hasher.Sum32())%SHARD_COUNT]
}

// Sets the given value under the specified key.
func (m *ConcurrentMap) Set(key <KEY>, value <VAL>) {
    // Get map shard.
    shard := m.GetShard(key)
    shard.Lock()
    defer shard.Unlock()
    shard.items[key] = value
}

[orcaman]

No good reason really. Your'e welcome to send a PR to fix this :-)