Skip to content

Refactor HardwareSerial.cpp for consistent use of PolledTimeout #6371

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
Aug 30, 2019

Conversation

dok-net
Copy link
Contributor

@dok-net dok-net commented Aug 1, 2019

After quick review comment, both of the two places in HardwareSerial.cpp shall use PolledTimeout - fixed by this PR.

@dok-net dok-net changed the title Refactor for consistency - use PolledTimeout throughout Refactor for consistency - (don't) use PolledTimeout throughout Aug 1, 2019
@dok-net dok-net changed the title Refactor for consistency - (don't) use PolledTimeout throughout Refactor for consistency - (don't) use PolledTimeout Aug 1, 2019
@dok-net dok-net force-pushed the hwser_consistency branch from f2b0682 to 70c904e Compare August 1, 2019 14:28
@dok-net dok-net changed the title Refactor for consistency - (don't) use PolledTimeout Refactor HardwareSerial.cpp for consistency - (don't) use PolledTimeout Aug 1, 2019
@dok-net dok-net force-pushed the hwser_consistency branch from 70c904e to 08da6d2 Compare August 2, 2019 13:39
@dok-net dok-net changed the title Refactor HardwareSerial.cpp for consistency - (don't) use PolledTimeout Refactor HardwareSerial.cpp for consistent use of PolledTimeout Aug 2, 2019
@dok-net dok-net force-pushed the hwser_consistency branch from 08da6d2 to 8195b17 Compare August 6, 2019 22:08
@dok-net dok-net force-pushed the hwser_consistency branch from 8195b17 to 244d656 Compare August 29, 2019 11:56
@devyte
Copy link
Collaborator

devyte commented Aug 30, 2019

Thanks @dok-net @d-a-v

@devyte devyte merged commit d04f768 into esp8266:master Aug 30, 2019
@dok-net dok-net deleted the hwser_consistency branch August 30, 2019 05:15
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.

3 participants