Skip to content

Comments

matcher: simplify concurrency#1

Open
crozzy wants to merge 1 commit intomainfrom
crozzy-simplify-Match-concurrency-pattern
Open

matcher: simplify concurrency#1
crozzy wants to merge 1 commit intomainfrom
crozzy-simplify-Match-concurrency-pattern

Conversation

@crozzy
Copy link
Owner

@crozzy crozzy commented Apr 19, 2021

Remove the error channel and the additional the sync needed to
administor it, reduce goroutines by one

Signed-off-by: crozzy joseph.crosland@gmail.com

@crozzy crozzy force-pushed the crozzy-simplify-Match-concurrency-pattern branch from 2abeb15 to 83d53b2 Compare April 19, 2021 14:11
Remove the error channel and the additional the sync needed to
administor it, also, use context for cancelation.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
@crozzy crozzy force-pushed the crozzy-simplify-Match-concurrency-pattern branch from 83d53b2 to 8db9171 Compare April 19, 2021 14:21
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.

1 participant