Skip to content

Fixes to pass validation with test suite #58

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 4 commits into from
Oct 29, 2015

Conversation

facchinm
Copy link
Member

This is needed to pass C++ validation in test suite

This is needed to pass C++ validation in test suite
@calvinatintel
Copy link
Contributor

@SidLeung @kmsywula @intel-larry Please review

Test suite failed on these functions
the Serial via USB port is not a real virtual cdc port (it still emulates an Hardware Serial), remove to avoid sketches getting confused and expect CDC flow control functions
needed to pass a very strict test
Porting of latest Due implementation
@facchinm facchinm changed the title Add new/delete implementations Fixes to pass validation with test suite Oct 28, 2015
@facchinm
Copy link
Member Author

I've added two more commits to address issues emerged with recently added tests.
Also changed the PR title to reflect the expanding collection of commits

@calvinatintel
Copy link
Contributor

@SidLeung @kmsywula @bigdinotech Please review

@bigdinotech
Copy link
Contributor

@calvinatintel looks good to me. +1

calvinatintel added a commit that referenced this pull request Oct 29, 2015
Fixes to pass validation with test suite
@calvinatintel calvinatintel merged commit 0ffb974 into arduino:master Oct 29, 2015
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