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

Add peek to TcpStream. #206

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add peek to TcpStream. #206

wants to merge 1 commit into from

Conversation

nickgian
Copy link

@nickgian nickgian commented Dec 30, 2024

Our codebase is using the peek functions that Turmoil does not model. It seemed easy to add so here's an attempt to add them for TCP.

@mcches
Copy link
Contributor

mcches commented Dec 30, 2024

I'll be out for a couple of days so my review will be delayed until later in the week. Adding @LucioFranco if you have availability.

I haven't looked yet, but we'll want to ensure #184 fits into whatever is added here.

@mcches mcches requested a review from LucioFranco December 30, 2024 18:33
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

Successfully merging this pull request may close these issues.

2 participants