Skip to content
This repository was archived by the owner on Jan 7, 2023. It is now read-only.

Conversation

@malsbat
Copy link
Contributor

@malsbat malsbat commented Apr 12, 2019

No description provided.

malsbat added 22 commits March 28, 2019 16:12
Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
This makes decoding bytes and strings consistent with all other CBOR
decode functions return DPS_ERR_EOD and do not modify the rxPos when
there is not enough data available.

Signed-off-by: Todd Malsbary <[email protected]>
In general, uv_pipe_getpeername() will return an empty name.  Looking
up remotes requires a unique name to work properly.

Signed-off-by: Todd Malsbary <[email protected]>
Signed-off-by: Todd Malsbary <[email protected]>
@malsbat malsbat requested a review from GregBurns April 12, 2019 21:20
}

static DPS_Status RegisterAndJoin(DPS_Node* node, char** linkText, int numLink,
static DPS_Status RegisterAndJoin(DPS_Node* node, char* network, char** linkText, int numLink,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be "const char* network"?

go/dps.go Outdated
static DPS_Status resolveAddress(DPS_Node* node, char* host, char* service, uintptr_t data) {
return DPS_ResolveAddress(node, host, service, onResolveAddressComplete, (void*)data);
static DPS_Status resolveAddress(DPS_Node* node, char* network, char* host, char* service, uintptr_t data) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"const char* network"??

Signed-off-by: Todd Malsbary <[email protected]>
@GregBurns GregBurns self-requested a review May 8, 2019 21:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants