Skip to content
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

Perf/cleanup closed #28

Merged
merged 13 commits into from
Mar 4, 2023
Merged

Perf/cleanup closed #28

merged 13 commits into from
Mar 4, 2023

Conversation

elementbound
Copy link
Owner

@elementbound elementbound commented Feb 26, 2023 •
edited
Loading

Description

  • Correspondences now emit close event when both ends close it
  • Peers free closed correspondences
  • Perf test is added to make sure memory doesn't leak when handling correspondences

Checklist

  • Documentation is up to date
  • Package exports are up to date
  • Versions are bumped as needed

Related issues

#23

@elementbound elementbound force-pushed the perf/cleanup-closed branch 2 times, most recently from e551024 to af7eb1b Compare February 26, 2023 14:36
@elementbound elementbound marked this pull request as ready for review February 26, 2023 14:52
@elementbound elementbound merged commit 34aa01e into main Mar 4, 2023
@elementbound elementbound deleted the perf/cleanup-closed branch March 4, 2023 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@elementbound