Skip to content

Commit d63bd30

Browse files
restyled-commitswoody-apple
authored andcommitted
Restyled by clang-format
1 parent e032c6e commit d63bd30

File tree

3 files changed

+55
-44
lines changed

3 files changed

+55
-44
lines changed

src/controller/CHIPDeviceController.cpp

+43-35
Original file line numberDiff line numberDiff line change
@@ -472,7 +472,7 @@ DeviceCommissioner::DeviceCommissioner() :
472472
DeviceCommissioner::~DeviceCommissioner()
473473
{
474474
#if CHIP_DEVICE_CONFIG_ENABLE_WIFIPAF
475-
SetChkObjValid((void *)this, ObjChkAction::Clear, nullptr);
475+
SetChkObjValid((void *) this, ObjChkAction::Clear, nullptr);
476476
#endif
477477
}
478478

@@ -833,7 +833,7 @@ CHIP_ERROR DeviceCommissioner::EstablishPASEConnection(NodeId remoteDeviceId, Re
833833
ExitNow(CHIP_ERROR_INTERNAL);
834834
}
835835
mRendezvousParametersForDeviceDiscoveredOverWiFiPAF = params;
836-
SetChkObjValid((void*)this, ObjChkAction::Set, nullptr);
836+
SetChkObjValid((void *) this, ObjChkAction::Set, nullptr);
837837
DeviceLayer::ConnectivityMgr().WiFiPAFConnect(params.GetSetupDiscriminator().value(), (void *) this,
838838
OnWiFiPAFSubscribeComplete, OnWiFiPAFSubscribeError);
839839
ExitNow(CHIP_NO_ERROR);
@@ -908,49 +908,55 @@ void DeviceCommissioner::OnDiscoveredDeviceOverBleError(void * appState, CHIP_ER
908908
#endif // CONFIG_NETWORK_LAYER_BLE
909909

910910
#if CHIP_DEVICE_CONFIG_ENABLE_WIFIPAF
911-
void DeviceCommissioner::SetChkObjValid(void * appObj, ObjChkAction action, bool* pIsObjValid)
911+
void DeviceCommissioner::SetChkObjValid(void * appObj, ObjChkAction action, bool * pIsObjValid)
912912
{
913-
static std::vector<void*> ObjVector;
913+
static std::vector<void *> ObjVector;
914914
bool IsObjValid = false;
915915

916-
switch (action) {
917-
case ObjChkAction::Set: {
918-
for (auto lt=ObjVector.begin(); lt!=ObjVector.end(); lt++)
916+
switch (action)
917+
{
918+
case ObjChkAction::Set: {
919+
for (auto lt = ObjVector.begin(); lt != ObjVector.end(); lt++)
920+
{
921+
if (*lt == appObj)
919922
{
920-
if (*lt == appObj) {
921-
IsObjValid = true;
922-
break;
923-
}
924-
}
925-
if (IsObjValid == false) {
926-
ObjVector.push_back(appObj);
927923
IsObjValid = true;
924+
break;
928925
}
929926
}
930-
break;
931-
case ObjChkAction::Check: {
932-
for (auto lt=ObjVector.begin(); lt!=ObjVector.end(); lt++)
927+
if (IsObjValid == false)
928+
{
929+
ObjVector.push_back(appObj);
930+
IsObjValid = true;
931+
}
932+
}
933+
break;
934+
case ObjChkAction::Check: {
935+
for (auto lt = ObjVector.begin(); lt != ObjVector.end(); lt++)
936+
{
937+
if (*lt == appObj)
933938
{
934-
if (*lt == appObj) {
935-
IsObjValid = true;
936-
break;
937-
}
939+
IsObjValid = true;
940+
break;
938941
}
939942
}
940-
break;
941-
case ObjChkAction::Clear: {
942-
for (auto lt=ObjVector.begin(); lt!=ObjVector.end(); lt++)
943+
}
944+
break;
945+
case ObjChkAction::Clear: {
946+
for (auto lt = ObjVector.begin(); lt != ObjVector.end(); lt++)
947+
{
948+
if (*lt == appObj)
943949
{
944-
if (*lt == appObj) {
945-
// Already existed in the list => Remove it
946-
ObjVector.erase(lt);
947-
break;
948-
}
950+
// Already existed in the list => Remove it
951+
ObjVector.erase(lt);
952+
break;
949953
}
950954
}
951-
break;
952955
}
953-
if (pIsObjValid != nullptr) {
956+
break;
957+
}
958+
if (pIsObjValid != nullptr)
959+
{
954960
*pIsObjValid = IsObjValid;
955961
}
956962
return;
@@ -960,12 +966,13 @@ void DeviceCommissioner::OnWiFiPAFSubscribeComplete(void * appState)
960966
{
961967
bool isObjValid;
962968
SetChkObjValid(appState, ObjChkAction::Check, &isObjValid);
963-
if (isObjValid == false) {
969+
if (isObjValid == false)
970+
{
964971
// The caller has been released.
965972
ChipLogError(Controller, "DeviceCommissioner has been destroyed!");
966973
return;
967974
}
968-
auto self = (DeviceCommissioner*) appState;
975+
auto self = (DeviceCommissioner *) appState;
969976
auto device = self->mDeviceInPASEEstablishment;
970977

971978
if (nullptr != device && device->GetDeviceTransportType() == Transport::Type::kWiFiPAF)
@@ -984,12 +991,13 @@ void DeviceCommissioner::OnWiFiPAFSubscribeError(void * appState, CHIP_ERROR err
984991
{
985992
bool isObjValid;
986993
SetChkObjValid(appState, ObjChkAction::Check, &isObjValid);
987-
if (isObjValid == false) {
994+
if (isObjValid == false)
995+
{
988996
// The caller has been released.
989997
ChipLogError(Controller, "DeviceCommissioner has been destroyed!");
990998
return;
991999
}
992-
auto self = (DeviceCommissioner*) appState;
1000+
auto self = (DeviceCommissioner *) appState;
9931001
auto device = self->mDeviceInPASEEstablishment;
9941002

9951003
if (nullptr != device && device->GetDeviceTransportType() == Transport::Type::kWiFiPAF)

src/controller/CHIPDeviceController.h

+2-1
Original file line numberDiff line numberDiff line change
@@ -822,7 +822,8 @@ class DLL_EXPORT DeviceCommissioner : public DeviceController,
822822
/* fireAndForget = */ true);
823823
}
824824
#if CHIP_DEVICE_CONFIG_ENABLE_WIFIPAF
825-
enum ObjChkAction{
825+
enum ObjChkAction
826+
{
826827
Set,
827828
Check,
828829
Clear

src/controller/SetUpCodePairer.cpp

+10-8
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ CHIP_ERROR GetPayload(const char * setUpCode, SetupPayload & payload)
6363
SetUpCodePairer::~SetUpCodePairer()
6464
{
6565
#if CHIP_DEVICE_CONFIG_ENABLE_WIFIPAF
66-
DeviceCommissioner::SetChkObjValid((void *)this, DeviceCommissioner::ObjChkAction::Clear, nullptr);
66+
DeviceCommissioner::SetChkObjValid((void *) this, DeviceCommissioner::ObjChkAction::Clear, nullptr);
6767
#endif
6868
}
6969

@@ -267,9 +267,9 @@ CHIP_ERROR SetUpCodePairer::StartDiscoverOverWiFiPAF(SetupPayload & payload)
267267
VerifyOrReturnError(mCommissioner != nullptr, CHIP_ERROR_INCORRECT_STATE);
268268
mWaitingForDiscovery[kWiFiPAFTransport] = true;
269269

270-
DeviceCommissioner::SetChkObjValid((void*)this, DeviceCommissioner::ObjChkAction::Set, nullptr);
271-
CHIP_ERROR err = DeviceLayer::ConnectivityMgr().WiFiPAFConnect(payload.discriminator, (void *) this,
272-
OnWiFiPAFSubscribeComplete, OnWiFiPAFSubscribeError);
270+
DeviceCommissioner::SetChkObjValid((void *) this, DeviceCommissioner::ObjChkAction::Set, nullptr);
271+
CHIP_ERROR err = DeviceLayer::ConnectivityMgr().WiFiPAFConnect(payload.discriminator, (void *) this, OnWiFiPAFSubscribeComplete,
272+
OnWiFiPAFSubscribeError);
273273
if (err != CHIP_NO_ERROR)
274274
{
275275
ChipLogError(Controller, "Commissioning discovery over WiFiPAF failed, err = %" CHIP_ERROR_FORMAT, err.Format());
@@ -401,25 +401,27 @@ void SetUpCodePairer::OnWiFiPAFSubscribeComplete(void * appState)
401401
{
402402
bool isObjValid;
403403
DeviceCommissioner::SetChkObjValid(appState, DeviceCommissioner::ObjChkAction::Check, &isObjValid);
404-
if (isObjValid == false) {
404+
if (isObjValid == false)
405+
{
405406
// The caller has been released.
406407
ChipLogError(Controller, "SetUpCodePairer has been destroyed!");
407408
return;
408409
}
409-
auto self = (SetUpCodePairer*) appState;
410+
auto self = (SetUpCodePairer *) appState;
410411
self->OnDiscoveredDeviceOverWifiPAF();
411412
}
412413

413414
void SetUpCodePairer::OnWiFiPAFSubscribeError(void * appState, CHIP_ERROR err)
414415
{
415416
bool isObjValid;
416417
DeviceCommissioner::SetChkObjValid(appState, DeviceCommissioner::ObjChkAction::Check, &isObjValid);
417-
if (isObjValid == false) {
418+
if (isObjValid == false)
419+
{
418420
// The caller has been released.
419421
ChipLogError(Controller, "SetUpCodePairer has been destroyed!");
420422
return;
421423
}
422-
auto self = (SetUpCodePairer*) appState;
424+
auto self = (SetUpCodePairer *) appState;
423425
self->OnWifiPAFDiscoveryError(err);
424426
}
425427
#endif

0 commit comments

Comments
 (0)