Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

ng_ndp: repaired debug for the ndp.c #3036

Merged
merged 1 commit into from
May 22, 2015
Merged

ng_ndp: repaired debug for the ndp.c #3036

merged 1 commit into from
May 22, 2015

Conversation

BytesGalore
Copy link
Member

Rationale: using ENABLE_DEBUG did not work because of missing string array and de-referencing mismatch passing the ipv6 address for string conversion.

@BytesGalore BytesGalore added Area: network Area: Networking Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation NSTF labels May 20, 2015
@miri64
Copy link
Member

miri64 commented May 20, 2015

Nice catch! ACK.

@miri64 miri64 mentioned this pull request May 20, 2015
Merged
miri64 added a commit that referenced this pull request May 22, 2015
@miri64
Merge pull request #3036 from BytesGalore/ng_ndp_debug_print_fixes
65fb2cf 
ng_ndp: repaired debug for the ndp.c
@miri64 miri64 merged commit 65fb2cf into RIOT-OS:master May 22, 2015
@BytesGalore BytesGalore deleted the ng_ndp_debug_print_fixes branch November 2, 2015 07:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@miri64 miri64

Labels
Area: network Area: Networking Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BytesGalore @miri64