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

Sync with upstream nng #2 #138

Merged
merged 10 commits into from
Feb 8, 2023
Merged

Sync with upstream nng #2 #138

merged 10 commits into from
Feb 8, 2023

Conversation

JaylinYu
Copy link
Member

@JaylinYu JaylinYu commented Feb 8, 2023

No description provided.

jimenezrick and others added 10 commits February 8, 2023 11:20
This function needs to be pretty much identical between PAIR v0 and
v1, it was missing just the call to release the pollable resources.
fixes #1543 by aborting tasks that may have been prepped, but not yet started.
Finish receive aio on tcp
pipe close
…ket on Windows (#1562)"

This reverts commit 1892e1d6d102d1fbd37e2c3bbb59dc35d81c8b33.
This reverts commit 8461c7207b440f5ba8c51b2236fcfa178f415a6f.
We try to move the msgq close up earler.  While here we can stop
dropping and reacquiring the lock -- this is likely left over
and may lead to races.
@codecov-commenter
Copy link

Codecov Report

Base: 55.74% // Head: 55.74% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (00f0519) compared to base (19b697a).
Patch coverage: 77.77% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
- Coverage   55.74%   55.74%   -0.01%     
==========================================
  Files          95       95              
  Lines       24887    24892       +5     
==========================================
+ Hits        13874    13876       +2     
- Misses      11013    11016       +3     
Impacted Files Coverage Δ
src/sp/transport/tcp/tcp.c 72.19% <33.33%> (+1.15%) ⬆️
src/core/dialer.c 82.71% <100.00%> (+0.14%) ⬆️
src/core/socket.c 85.89% <100.00%> (-0.04%) ⬇️
src/sp/protocol/pair0/pair.c 90.27% <100.00%> (+0.07%) ⬆️
src/platform/posix/posix_tcpconn.c 75.00% <0.00%> (-4.75%) ⬇️
src/sp/transport/inproc/inproc.c 78.35% <0.00%> (-1.87%) ⬇️
src/core/aio.c 91.89% <0.00%> (-1.21%) ⬇️
src/sp/protocol/pair1/pair.c 93.27% <0.00%> (-0.62%) ⬇️
src/platform/posix/posix_ipcconn.c 56.16% <0.00%> (-0.46%) ⬇️
src/core/stats.c 81.70% <0.00%> (-0.43%) ⬇️
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@JaylinYu JaylinYu merged commit ad52c57 into main Feb 8, 2023
@JaylinYu JaylinYu deleted the jaylin/nanosdk branch February 8, 2023 03:25
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.

6 participants