Use atomic.Bool for client.closed to reduce lock contention #233

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

Closed

cppcoffee wants to merge 1 commit into pion:master from cppcoffee:master

Closed

Use atomic.Bool for client.closed to reduce lock contention #233

Reduce lock contention with atomic.Boo

Codecov / codecov/patch failed Jun 29, 2025 in 0s

0.00% of diff hit (target 70.00%)

View this Pull Request on Codecov

0.00% of diff hit (target 70.00%)

Annotations

Check warning on line 349 in client.go

codecov / codecov/patch

client.go#L349

Added line #L349 was not covered by tests

Check warning on line 353 in client.go

codecov / codecov/patch

client.go#L352-L353

Added lines #L352 - L353 were not covered by tests

Check warning on line 489 in client.go

codecov / codecov/patch

client.go#L489

Added line #L489 was not covered by tests

Check warning on line 492 in client.go

codecov / codecov/patch

client.go#L492

Added line #L492 was not covered by tests

Check warning on line 623 in client.go

codecov / codecov/patch

client.go#L623

Added line #L623 was not covered by tests

Check warning on line 626 in client.go

codecov / codecov/patch

client.go#L626

Added line #L626 was not covered by tests

Check warning on line 702 in client.go

codecov / codecov/patch

client.go#L702

Added line #L702 was not covered by tests

View more details on Codecov