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

network_layer: remove last occurences of vtimer #4186

Merged
merged 4 commits into from
Nov 2, 2015

Conversation

cgundogan
Copy link
Member

This PR removes further occurences of vtimer by replacing them with xtimer in gnrc_icmpv6, gnrc_ipv6, gnrc_ndp and gnrc_sixlowpan

@cgundogan cgundogan added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking labels Oct 28, 2015
@cgundogan cgundogan added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 28, 2015
@miri64
Copy link
Member

miri64 commented Oct 29, 2015

Can you also replace the last occurrences of timex_t while you are at it?

@@ -24,7 +24,6 @@
#include "net/gnrc/sixlowpan/nd.h"
#include "thread.h"
#include "timex.h"
#include "vtimer.h"
Copy link
Member

Choose a reason for hiding this comment

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

+#include "xtimer.h"

?

(below too?)

Copy link
Member Author

Choose a reason for hiding this comment

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

xtimer.h gets included by other header files AFAICS and is not needed here

Copy link
Member

Choose a reason for hiding this comment

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

Transitive includes: Guaranteed future weird build errors ;).

Copy link
Member Author

Choose a reason for hiding this comment

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

addressed

@cgundogan
Copy link
Member Author

Can you also replace the last occurrences of timex_t while you are at it?

sure, will do that in a follow-up PR

@cgundogan
Copy link
Member Author

@authmillenon addressed your comment

@miri64
Copy link
Member

miri64 commented Nov 2, 2015

ACK.

@miri64 miri64 added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Nov 2, 2015
@cgundogan cgundogan force-pushed the pr/ipv6/last_vtimers branch from 7d5d7fc to 5fb1041 Compare November 2, 2015 14:02
@cgundogan
Copy link
Member Author

squashed

@cgundogan cgundogan removed the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Nov 2, 2015
@cgundogan
Copy link
Member Author

travis agrees - GO

cgundogan added a commit that referenced this pull request Nov 2, 2015
network_layer: remove last occurences of vtimer
@cgundogan cgundogan merged commit 42c5666 into RIOT-OS:master Nov 2, 2015
@cgundogan cgundogan deleted the pr/ipv6/last_vtimers branch November 2, 2015 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR 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