Skip to content

Added missing function call to serialEvent example #3203

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

Merged
merged 1 commit into from
May 21, 2015

Conversation

agdl
Copy link
Member

@agdl agdl commented May 21, 2015

In the example is stated that the function is run between one loop and the next, but actually the call to the function was missing.
The comment also state that the response can be delayed using a delay in the loop,so I think that the way it should be is so by only
adding a call to the function serialEvent as first operation in the loop. I so added this call.

@agdl agdl mentioned this pull request May 21, 2015
In the example is stated that the function is run between one loop and the next, but actually the call to the function was missing.
The comment also state that the response can be delayed using a delay in the loop,so I think that the way it should be is so by only
adding a call to the function serialEvent as first operation in the loop. I so added this call.
@agdl agdl force-pushed the SerialEventFix branch from e19947f to 4648330 Compare May 21, 2015 08:16
@agdl agdl changed the title This fixes #3179 Added missing function call to serialEvent example May 21, 2015
ffissore added a commit that referenced this pull request May 21, 2015
Added missing function call to serialEvent example
@ffissore ffissore merged commit 2e1696f into arduino:master May 21, 2015
@ffissore ffissore self-assigned this May 21, 2015
@ffissore ffissore added this to the Release 1.6.5 milestone May 21, 2015
@agdl agdl deleted the SerialEventFix branch March 18, 2016 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants