abiosoft/semaphore

data race when running tests

Closed this issue · 2 comments

Run with "go test -race ./..."

==================
WARNING: DATA RACE
Read at 0x00c4200742d8 by goroutine 7:
  github.com/kevinburke/semaphore.(*Semaphore).Release()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore.go:95 +0xde
  github.com/kevinburke/semaphore.(*Semaphore).ReleaseMany()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore.go:104 +0x61
  github.com/kevinburke/semaphore.aq()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore_test.go:44 +0x37e

Previous write at 0x00c4200742d8 by goroutine 12:
  github.com/kevinburke/semaphore.(*Semaphore).AcquireMany()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore.go:57 +0x103
  github.com/kevinburke/semaphore.aq()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore_test.go:41 +0x1d9

Goroutine 7 (running) created at:
  github.com/kevinburke/semaphore.Test()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore_test.go:18 +0xae
  testing.tRunner()
      /Users/kevin/go/src/testing/testing.go:746 +0x16c

Goroutine 12 (running) created at:
  github.com/kevinburke/semaphore.Test()
      /Users/kevin/src/github.com/kevinburke/semaphore/semaphore_test.go:18 +0xae
  testing.tRunner()
      /Users/kevin/go/src/testing/testing.go:746 +0x16c

Thanks for reporting this.

Fixed