New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use new sync Map in Go 1.9 #269

Open

jcscottiii opened this issue Aug 25, 2017 · 1 comment

Member

jcscottiii commented Aug 25, 2017

Previously, we created maps with a manually maintained mutex:
Example here:

compliance-masonry/lib/standards.go

Lines 11 to 14 in 40b7371

    
           type standardsMap struct { 
        
           	mapping map[string]common.Standard 
        
           	sync.RWMutex 
        
           }

(there's more cases throughout the code too)

Now, with Go 1.9, there's a map that's safe for concurrent access:
https://golang.org/doc/go1.9#sync-map

Member Author

jcscottiii commented Aug 25, 2017

Should do #262 first

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