Skip to content

Cellular: AT_CellularContext disconnect in non-blocking mode #10656

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 31, 2019
Merged

Cellular: AT_CellularContext disconnect in non-blocking mode #10656

merged 1 commit into from
May 31, 2019

Conversation

jarvte
Copy link
Contributor

@jarvte jarvte commented May 24, 2019 •
edited
Loading

Description

Disconnect was supporting only blocking mode.

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@mirelachirica

Release Notes

@ciarmcom ciarmcom requested review from mirelachirica and a team May 24, 2019 13:01
@ciarmcom
Copy link
Member

@jarvte, thank you for your changes.
@mirelachirica @ARMmbed/mbed-os-wan @ARMmbed/mbed-os-maintainers please review.

@jarvte jarvte force-pushed the fix_cellularctx_destr branch from 1002adf to 753777b Compare May 27, 2019 09:05
@jarvte jarvte changed the title Cellular: fix AT_CellularContext destructor in non-blocking mode Cellular: AT_CellularContext disconnect in non-blocking mode May 27, 2019
Cellular: AT_CellularContext disconnect in non-blocking mode
779fb6d 
Disconnect was supporting only blocking mode.
@jarvte jarvte force-pushed the fix_cellularctx_destr branch from 753777b to 779fb6d Compare May 27, 2019 09:23
@mbed-ci
Copy link

mbed-ci commented May 29, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@0xc0170 0xc0170 merged commit 523ad23 into ARMmbed:master May 31, 2019
@jarvte jarvte deleted the fix_cellularctx_destr branch June 3, 2019 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

@mirelachirica mirelachirica mirelachirica approved these changes

Assignees
No one assigned
Labels
release-version: 5.13.1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jarvte @ciarmcom @mbed-ci @0xc0170 @mirelachirica