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

send token on error message. #2

Merged
merged 1 commit into from Jul 24, 2014
Merged

Conversation

sbernard31
Copy link
Contributor

See original pull request here: intel/liblwm2m#31

@dnav
Copy link
Contributor

dnav commented Jul 23, 2014

Shouldn't this be
response->code = coap_error_code;
instead of message->code ?

Also could you put the else on its own line just to be consistent with the file formatting ?

Thanks

@sbernard31
Copy link
Contributor Author

I fix the formatting problem. (sry for that)
And for the response->code VS message->code, you're right but I take some time to understand why it was working with message->code and it seems response->code is already set in handle_request(), so I remove the line.

dnav pushed a commit that referenced this pull request Jul 24, 2014
send token on error message.
@dnav dnav merged commit 620b63e into eclipse-wakaama:master Jul 24, 2014
@sbernard31 sbernard31 deleted the sendtoken branch August 1, 2014 12:38
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.

None yet

2 participants