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

Check signature of t.connect().start() in checkSender #16

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Check signature of t.connect().start() in checkSender #16

wants to merge 4 commits into from

Conversation

nordlow
Copy link
Contributor

@nordlow nordlow commented May 11, 2021

No description provided.

@nordlow nordlow changed the title Check return value of connect() in checkSender Check signature of connect().start() in checkSender May 11, 2021
@nordlow nordlow changed the title Check signature of connect().start() in checkSender Check signature of t.connect().start() in checkSender May 11, 2021
@nordlow nordlow force-pushed the better-checkSender branch 2 times, most recently from 16a8b17 to 9939321 Compare May 11, 2021 11:47
@skoppe
Copy link
Collaborator

skoppe commented May 11, 2021 •
edited
Loading

Maybe we can use the opportunity to simplify the static branches a bit, by moving them down into the Receiver object

@nordlow
Copy link
Contributor Author

nordlow commented May 11, 2021 •
edited
Loading

Maybe we can use the opportunity to simplify the static branches a bit, by moving them down into the Receiver object

Does this mean defining two checkSenders one with Value being void and one being non-void? If not, I don't follow.

@skoppe skoppe force-pushed the master branch 2 times, most recently from d863045 to 29e41b4 Compare July 22, 2021 15:06
@skoppe skoppe force-pushed the master branch 16 times, most recently from 9483e7d to f671b4f Compare August 16, 2021 14:28
@skoppe skoppe force-pushed the master branch 2 times, most recently from 3b6b14e to 9b3c09b Compare September 18, 2024 23:16
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.
2 participants
@nordlow @skoppe