From 1279c20ce6a7c97a51ef1532937c72d65455579f Mon Sep 17 00:00:00 2001 From: Kimmo Vaisanen Date: Fri, 4 Oct 2019 10:29:56 +0300 Subject: [PATCH] Cellular: Change visibility of _is_connected as protected AT_CellularContext::do_connect() is a virtual API and therefore can be overwritten in inherited class. The problem was that it sets AT_CellularContext::_is_connected flag but earlier it was set as private member making it impossible to set in overwritten do_connect() method. This commit fixes the problem by changing _is_connected as protected enabling its use in inherited class. --- features/cellular/framework/AT/AT_CellularContext.cpp | 2 +- features/cellular/framework/AT/AT_CellularContext.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/features/cellular/framework/AT/AT_CellularContext.cpp b/features/cellular/framework/AT/AT_CellularContext.cpp index e0e192e8fb..9608ae4073 100644 --- a/features/cellular/framework/AT/AT_CellularContext.cpp +++ b/features/cellular/framework/AT/AT_CellularContext.cpp @@ -47,7 +47,7 @@ using namespace mbed; using namespace rtos; AT_CellularContext::AT_CellularContext(ATHandler &at, CellularDevice *device, const char *apn, bool cp_req, bool nonip_req) : - AT_CellularBase(at), _is_connected(false), _current_op(OP_INVALID), _fh(0), _cp_req(cp_req) + AT_CellularBase(at), _current_op(OP_INVALID), _fh(0), _cp_req(cp_req), _is_connected(false) { tr_info("New CellularContext %s (%p)", apn ? apn : "", this); _nonip_req = nonip_req; diff --git a/features/cellular/framework/AT/AT_CellularContext.h b/features/cellular/framework/AT/AT_CellularContext.h index 173a836a3c..5ccb5f2051 100644 --- a/features/cellular/framework/AT/AT_CellularContext.h +++ b/features/cellular/framework/AT/AT_CellularContext.h @@ -121,7 +121,6 @@ private: void do_disconnect(); void set_cid(int cid); private: - bool _is_connected; ContextOperation _current_op; FileHandle *_fh; rtos::Semaphore _semaphore; @@ -131,6 +130,7 @@ protected: char _found_apn[MAX_APN_LENGTH]; // flag indicating if CP was requested to be setup bool _cp_req; + bool _is_connected; }; } // namespace mbed