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.

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

send-message-batch's defaults leak resources #35

Open
ecpeterson opened this issue Jun 28, 2021 · 0 comments
Open

send-message-batch's defaults leak resources #35

ecpeterson opened this issue Jun 28, 2021 · 0 comments

Comments

@ecpeterson
Copy link
Contributor

When the convenience keyword :replies? is flagged, send-message-batch registers a mailbox per message to use as a reply repository.

This keyword currently defaults to t, since send-message-batch is almost everywhere paired with with-replies, which does the job of closing out the corresponding mailbox. On the other hand, this means that an inattentive programmer using send-message-batch away from with-replies runs a high risk of leaking resources.

This bit me recently, so I'm wondering if we should investigate this usability change.

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

No branches or pull requests

1 participant