Closed bollhals closed 6 days ago
@bollhals I converted this PR to a draft. Feel free to click "Ready for review" when you're done.
Thank you for this contribution!
@bollhals I converted this PR to a draft. Feel free to click "Ready for review" when you're done.
Thank you for this contribution!
It's ready. It was just some whitespace formatting issue. (My settings are different than what's expected here)
Nice!
Thank you! I will merge once CI passes and will release RC.3
Just FYI, I have currently a bit of spare time (again after some busy years^^) and plan to push a few updates. (Up to you to include it them in whatever version you see fit 👍 (Another RC or 7.x or even 8))
@bollhals thank you, I will keep an eye out. I would like to get version 7 released soon but am happy to wait on your contributions.
Proposed Changes
Changes the InboundFrame to a class instead of a readonly struct. There's only ever one frame active per connection, hence it can be a class that is allocated once and then reused.
benefits
in
).Types of Changes
Checklist
CONTRIBUTING.md
documentFurther Comments
There's a bugfix in here as well, if wanted, I can submit that separately. see my comments.