Skip to content

tests/driver_servo/main.c: fix var len#7852

Merged
kYc0o merged 1 commit intoRIOT-OS:masterfrom
kYc0o:fix_servo_var
Oct 26, 2017
Merged

tests/driver_servo/main.c: fix var len#7852
kYc0o merged 1 commit intoRIOT-OS:masterfrom
kYc0o:fix_servo_var

Conversation

@kYc0o
Copy link
Copy Markdown
Contributor

@kYc0o kYc0o commented Oct 26, 2017

While compiling tests/driver_servo with clang in OS X it discovers several minor issues.

This PR fixes them.

@kYc0o kYc0o added Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Area: tests Area: tests and testing framework labels Oct 26, 2017
@kYc0o kYc0o added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 26, 2017
Copy link
Copy Markdown
Member

@jnohlgard jnohlgard left a comment

Choose a reason for hiding this comment

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

If this fixes a compilation error I don't mind. It should not affect the behavior of the test in any way.

@kYc0o
Copy link
Copy Markdown
Contributor Author

kYc0o commented Oct 26, 2017

Yes it's just compilation issues raised by clang. My efforts are towards #1121.

@kYc0o kYc0o merged commit 250fce1 into RIOT-OS:master Oct 26, 2017
@kYc0o kYc0o mentioned this pull request Oct 26, 2017
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants