Skip to content

rpl: expose global rpl_dodags variable#2622

Merged
OlegHahm merged 1 commit intoRIOT-OS:masterfrom
cgundogan:rpl_get_dodags_function
Mar 19, 2015
Merged

rpl: expose global rpl_dodags variable#2622
OlegHahm merged 1 commit intoRIOT-OS:masterfrom
cgundogan:rpl_get_dodags_function

Conversation

@cgundogan
Copy link
Copy Markdown
Member

It is necessary to be able to read the dodags array outside of rpl_dodag.c

@cgundogan cgundogan added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking labels Mar 17, 2015
@BytesGalore BytesGalore mentioned this pull request Mar 17, 2015
21 tasks
@OlegHahm
Copy link
Copy Markdown
Member

I'd like to quote @Kijewski: "Why not just expose the variable? This is not Java with getters and setters. :)
The result should the const if you prefer the getter." (#1228 (comment))

@cgundogan cgundogan force-pushed the rpl_get_dodags_function branch from 84f70e2 to d7722c1 Compare March 19, 2015 10:04
@cgundogan
Copy link
Copy Markdown
Member Author

addressed @OlegHahm's comment and rebased to master

@cgundogan cgundogan changed the title rpl: add function to retrieve all dodags rpl: expose global rpl_dodags variable Mar 19, 2015
@OlegHahm
Copy link
Copy Markdown
Member

Wonderful. ACK and go!

OlegHahm added a commit that referenced this pull request Mar 19, 2015
rpl: expose global rpl_dodags variable
@OlegHahm OlegHahm merged commit bfe2c9a into RIOT-OS:master Mar 19, 2015
@cgundogan cgundogan deleted the rpl_get_dodags_function branch March 19, 2015 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants