/amz

Make counter thread safe #61

Merged
merged 1 commit into from Aug 20, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@dooferlad @dimitern
@dooferlad

dooferlad commented Aug 20, 2015

ect/ec2test/helper.go counter was not being used in a thread safe manner, so now it is a struct with next, get and reset functions. The old next function just incremented an integer and returned the old value. Now accessing by next, get and reset are all atomic.
@dooferlad
ect/ec2test/helper.go counter was not being used in a thread safe man… 
…ner, so now it is a struct with next, get and reset functions. The old next function just incremented an integer and returned the old value. Now accessing by next, get and reset are all atomic.
8654fa0
@dimitern
Member

dimitern commented Aug 20, 2015

Nice! Thanks, I'll merge this.

dimitern pushed a commit that referenced this pull request Aug 20, 2015

Merge pull request #61 from dooferlad/fix-counter-race 
Make counter thread safe
bff3a09

@dimitern dimitern merged commit bff3a09 into go-amz:v3 Aug 20, 2015
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@dooferlad dooferlad deleted the dooferlad:fix-counter-race branch Aug 20, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment