Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

`counters*` atom should be a map of keys -> counter atoms #216

Open
neatonk opened this Issue · 1 comment

2 participants

@neatonk
Collaborator

See https://github.com/overtone/overtone/blob/master/src/overtone/libs/counters.clj#L6

(defonce counters* (atom {}))

Using a single atom containing a map of keyed counters for :node, :bus, :buffer creates unnecessary contention around the counters* atom. Only one counter can be updated at a time. We should probably create an atom for each key.

@samaaron
Owner

I'm not sure it's a huge bottle neck, given that the operation is just incing a long. However let's look into this post 0.8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.