Skip to content

Conversation

@DorZion
Copy link
Contributor

@DorZion DorZion commented Jan 12, 2020

Seems redundant and nobody uses the value of Executors.newCachedThreadPool, unless I'm missing something.

Seems redundant and nobody uses the value of `Executors.newCachedThreadPool`, unless I'm missing something.
Removed cached thread pool creation
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 83.937% when pulling 4284026 on DorZion:master into 98a70af on Coreoz:master.

@amanteaux
Copy link
Member

Well spotted! It must be an artefact left during a debugging session. I feel ashamed to have let this slipped!

@amanteaux
Copy link
Member

Looks like GitHub does not want me to merge this from my phone. I will do that tomorrow.

@amanteaux amanteaux merged commit e9ea465 into Coreoz:master Jan 13, 2020
@amanteaux
Copy link
Member

Released in Wisp 2.2.0

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants