Skip to content
This repository was archived by the owner on Nov 13, 2025. It is now read-only.

Conversation

@ArgoZhang
Copy link
Member

Link issues

fixes #27

Summary By Copilot

Regression?

  • Yes
  • No

Risk

  • High
  • Medium
  • Low

Verification

  • Manual (required)
  • Automated

Packaging changes reviewed?

  • Yes
  • No
  • N/A

☑️ Self Check before Merge

⚠️ Please check all items below before review. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • Merge the latest code from the main branch

Copilot AI review requested due to automatic review settings September 17, 2025 02:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR optimizes memory allocation by reusing Sender and Receiver instances instead of creating new ones for each operation. The change eliminates the overhead of repeatedly allocating and disposing these objects during socket communications.

  • Replaced per-operation instantiation with reusable instance fields
  • Added proper disposal of reused instances in the cleanup method
  • Modified null-checking logic to accommodate the new reuse pattern

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ArgoZhang ArgoZhang merged commit 0d5ccd2 into master Sep 17, 2025
1 check passed
@ArgoZhang ArgoZhang deleted the refactor-mem branch September 17, 2025 02:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: reuse Sender/Receiver reduce memory allocation

2 participants