Skip to content

xtimer: small cleanups#4181

Merged
jnohlgard merged 7 commits intoRIOT-OS:masterfrom
jnohlgard:pr/xtimer-small-cleanups
Oct 29, 2015
Merged

xtimer: small cleanups#4181
jnohlgard merged 7 commits intoRIOT-OS:masterfrom
jnohlgard:pr/xtimer-small-cleanups

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

This PR is only a clean up of xtimer code, no added functionality or bugs fixed. This clean up is necessary to make future xtimer PRs more readable and easier to follow.

@jnohlgard jnohlgard added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Area: timers Area: timer subsystems labels Oct 28, 2015
@kaspar030
Copy link
Copy Markdown
Contributor

ACK.

@kaspar030 kaspar030 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
@kaspar030
Copy link
Copy Markdown
Contributor

@gebart Would some squashing make sense?

@jnohlgard
Copy link
Copy Markdown
Member Author

@kaspar030 I can squash them if you want, what should I do with the commit message? Make a single list from the current (split) commit messages?

@kaspar030
Copy link
Copy Markdown
Contributor

@gebart Actually the commits are fine as is. Lets skip the squashing. :)

@jnohlgard
Copy link
Copy Markdown
Member Author

Travis is happy -> go

jnohlgard pushed a commit that referenced this pull request Oct 29, 2015
@jnohlgard jnohlgard merged commit 8b378d6 into RIOT-OS:master Oct 29, 2015
@jnohlgard jnohlgard deleted the pr/xtimer-small-cleanups branch October 29, 2015 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: timers Area: timer subsystems CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants