Skip to content

netif: add functions to parse and output hardware addresses#2705

Merged
Lotterleben merged 2 commits intoRIOT-OS:masterfrom
miri64:netif/feat/addr-str
Apr 1, 2015
Merged

netif: add functions to parse and output hardware addresses#2705
Lotterleben merged 2 commits intoRIOT-OS:masterfrom
miri64:netif/feat/addr-str

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Mar 24, 2015

Adds address parsing and string-conversion functions for hardware addresses in a similar way as I did for ng_ipv6_addr.

@miri64 miri64 added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation NSTF labels Mar 24, 2015
This was referenced Mar 24, 2015
Merged
Merged
haukepetersen
haukepetersen reviewed Mar 24, 2015
View reviewed changes
sys/include/net/ng_netif.h Outdated
Copy link
Copy Markdown
Contributor

@haukepetersen haukepetersen Mar 24, 2015

Choose a reason for hiding this comment

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

maybe better: "Converts a hardware address to a \0 terminated string"

Copy link
Copy Markdown
Member Author

@miri64 miri64 Mar 24, 2015

Choose a reason for hiding this comment

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

This sounds misleading: a '\0' terminated string can also just be the bytes of the address (not converted to hex strings!) + an appended '\0'

Copy link
Copy Markdown
Contributor

@haukepetersen haukepetersen Mar 24, 2015

Choose a reason for hiding this comment

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

hm, ok. Just wanted to stress that the string is NULL terminated to be used in string functions (as printf("%s", ...)

Copy link
Copy Markdown
Member Author

@miri64 miri64 Mar 24, 2015

Choose a reason for hiding this comment

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

Okay, I get your point will do s/format/string/. Does that suffice?

Copy link
Copy Markdown
Member Author

@miri64 miri64 Mar 24, 2015

Choose a reason for hiding this comment

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

And done.

@miri64 miri64 force-pushed the netif/feat/addr-str branch from 789b6d3 to 04a895e Compare March 29, 2015 13:17
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 29, 2015

Rebased to current master

@miri64 miri64 force-pushed the netif/feat/addr-str branch from 04a895e to 325b128 Compare March 30, 2015 01:10
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 30, 2015

Rebased to current master

@miri64 miri64 force-pushed the netif/feat/addr-str branch from 325b128 to fe50e2a Compare March 30, 2015 02:00
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 30, 2015

Rebased to current master (again Oo)

@OlegHahm OlegHahm force-pushed the master branch 3 times, most recently from 9f184dd to 45554bf Compare March 31, 2015 13:01
@miri64 miri64 assigned Lotterleben and unassigned haukepetersen Mar 31, 2015
Lotterleben
Lotterleben reviewed Apr 1, 2015
View reviewed changes
sys/include/net/ng_netif.h Outdated
Copy link
Copy Markdown
Member

@Lotterleben Lotterleben Apr 1, 2015

Choose a reason for hiding this comment

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

seperate -> separate

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Apr 1, 2015

Addressed comments and optimized netif_addr_from_str a little ;-)

@Lotterleben
Copy link
Copy Markdown
Member

Lotterleben commented Apr 1, 2015

thanks :)
I'm still not happy with the inconsistency of the return values, but I guess this compromise in favor of usability (ng_netif_addr_to_str ()) and resourcefulness (ng_netif_addr_from_str ()) is inevitable...
Let's squash and see what Travis says.

@miri64 miri64 force-pushed the netif/feat/addr-str branch from db0b14e to 1a0d97f Compare April 1, 2015 14:24
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Apr 1, 2015

Squashed.

@Lotterleben
Copy link
Copy Markdown
Member

Lotterleben commented Apr 1, 2015

Kicked Travis...

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Apr 1, 2015

And it's done

@Lotterleben
Copy link
Copy Markdown
Member

Lotterleben commented Apr 1, 2015

Here we go... ACK and go! :)

Lotterleben added a commit that referenced this pull request Apr 1, 2015
@Lotterleben
Merge pull request #2705 from authmillenon/netif/feat/addr-str
40acfa7 
netif: add functions to parse and output hardware addresses
@Lotterleben Lotterleben merged commit 40acfa7 into RIOT-OS:master Apr 1, 2015
@miri64 miri64 deleted the netif/feat/addr-str branch April 1, 2015 23:05
@miri64 miri64 added the Area: network Area: Networking label Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Lotterleben Lotterleben

Labels

Area: network Area: Networking Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@miri64 @Lotterleben @haukepetersen