search

docker / go-events

Composable event distribution for Go
Apache License 2.0
131 stars 22 forks source link

all: fix sync.Once copy #20

Closed LK4D4 closed 8 years ago

LK4D4 commented 8 years ago

We're doing object copy on String call, but it makes race detector angry: copy of sync.Once and changing its state at the same time. Also, it fixes three vet warnings about passing lock by value.

ping @aaronlehmann @stevvooe

aaronlehmann commented 8 years ago

Nice catch. Hopefully this finally fixes the race detector issues.

stevvooe commented 8 years ago

LGTM