Skip to content

drivers/encx24j600/encx24j600.c : fix counter var length#7858

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

drivers/encx24j600/encx24j600.c : fix counter var length#7858
aabadie merged 1 commit intoRIOT-OS:masterfrom
kYc0o:fix_encx24j600_comp

Conversation

@kYc0o
Copy link
Copy Markdown
Contributor

@kYc0o kYc0o commented Oct 26, 2017

While compiling tests/driver_encx24j600 with clang in OS X it discovers a minor issue.

This PR fixes it.

@kYc0o kYc0o added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: drivers Area: Device drivers Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 26, 2017
@kYc0o kYc0o requested a review from kaspar030 October 26, 2017 00:29
Copy link
Copy Markdown
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

ACK

@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Oct 26, 2017

and go

@aabadie aabadie merged commit 4de88d3 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: drivers Area: Device drivers 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 Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants