/subunit-go

Added a mutex to the writer. #3

Merged
merged 2 commits into from Jan 4, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@elopio @fgimenez
@elopio
Contributor

elopio commented Dec 15, 2015

No description provided.

@fgimenez

fgimenez commented Dec 16, 2015

👍 being so close the lock and the unlock makes no sense to use a defer
@elopio
Contributor

elopio commented Jan 4, 2016

oh wait, actually I wanted a defer, but forgot about it :)
I'll make the change and merge.

elopio added a commit that referenced this pull request Jan 4, 2016

@elopio
Merge pull request #3 from elopio/mutex 
Added a mutex to the writer.
30fcecc

@elopio elopio merged commit 30fcecc into testing-cabal:master Jan 4, 2016
1 check passed

1 check passed

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

@elopio elopio deleted the elopio:mutex branch Jan 4, 2016

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