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

Changed error print to assert #5069

Merged
merged 1 commit into from Sep 15, 2017

Conversation

Projects
None yet
4 participants
@deepikabhavnani
Contributor

deepikabhavnani commented Sep 11, 2017

Changed error print to assert function, since using malloc instead of new.

This PR is to close review comment related to emac_interface_t in PR #4938

https://github.com/ARMmbed/mbed-os/pull/4938/files/6bd28ecb685542237808f9e009dbb9510372d14c#diff-5cc67fcf86ead5e18c66e794b692d326L220

@deepikabhavnani deepikabhavnani requested a review from sg- Sep 14, 2017

@deepikabhavnani

This comment has been minimized.

Contributor

deepikabhavnani commented Sep 14, 2017

/morph test

@sg-

sg- approved these changes Sep 14, 2017

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 14, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1307

All builds and test passed!

@theotherjimmy theotherjimmy merged commit aeda7f7 into ARMmbed:master Sep 15, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@deepikabhavnani deepikabhavnani deleted the deepikabhavnani:assert_added branch Sep 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment