Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This branch contains the updates to support the NNG v2.0 API.
CI will currently fail as I have not yet updated the bundled versions of mbedtls + NNG.
/usr/local/lib
Internal changes:
nng_close()
->nng_socket_close()
,nng_recv_aio()
->nng_socket_recv()
,nng_send_aio()
->nng_socket_send()
nng_init(NULL)
is called from package load routine. This allowsnng_fini()
to be safely called on package unload (previously avoided)http_conn
struct instead of request/responsesudp://
transportRevised behaviour:
opt()
/opt<-()
now no longer support string / uint64 / NULL option typesparse_url()
now returns a character vector of length 7 rather than 10pipe_notify()
must be called before the socket listen / dial eventsTests have been updated for above behavioural changes and test clean
TODOs:
opt()
, return options on the stream dialer / listener rather than the streamSupport for new features will be implemented in separate PRs that target this one.
FYI @lionel-