Skip to content

Commit 090d9b7

Browse files
restyled-commitsyunhanw-google
authored andcommitted
Restyled by clang-format
1 parent c488168 commit 090d9b7

File tree

4 files changed

+13
-13
lines changed

4 files changed

+13
-13
lines changed

src/app/ReadClient.cpp

+4-4
Original file line numberDiff line numberDiff line change
@@ -490,8 +490,8 @@ void ReadClient::OnActiveModeNotification()
490490
// called, either mEventPathParamsListSize or mAttributePathParamsListSize is not 0.
491491
VerifyOrReturn(mReadPrepareParams.mEventPathParamsListSize != 0 || mReadPrepareParams.mAttributePathParamsListSize != 0);
492492

493-
// If mCatsMatchCheckIn is true, it means cats used in icd registration matches with the one in current subscription, OnActiveModeNotification
494-
// continues to revive the subscription as needed, otherwise, do nothing.
493+
// If mCatsMatchCheckIn is true, it means cats used in icd registration matches with the one in current subscription,
494+
// OnActiveModeNotification continues to revive the subscription as needed, otherwise, do nothing.
495495
VerifyOrReturn(mReadPrepareParams.mCatsMatchCheckIn);
496496

497497
// When we reach here, the subscription definitely exceeded the liveness timeout. Just continue the unfinished resubscription
@@ -503,8 +503,8 @@ void ReadClient::OnActiveModeNotification()
503503
}
504504

505505
// If the server sends out check-in message, and there is a tracked active subscription in client side at the same time, it
506-
// means current client does not realize this tracked subscription has gone, and we should forcibly timeout current subscription, and
507-
// schedule a new one.
506+
// means current client does not realize this tracked subscription has gone, and we should forcibly timeout current
507+
// subscription, and schedule a new one.
508508
if (!mIsResubscriptionScheduled)
509509
{
510510
// Closing will ultimately trigger ScheduleResubscription with the aReestablishCASE argument set to true, effectively

src/app/ReadClient.h

+3-3
Original file line numberDiff line numberDiff line change
@@ -357,9 +357,9 @@ class ReadClient : public Messaging::ExchangeDelegate
357357
* If the subscription is not in the `InactiveICDSubscription` state, this function will do nothing. So it is always safe to
358358
* call this function when a check-in message is received.
359359
*
360-
* If the server sends out check-in message, and there is a active tracked active subscription in client side at the same time, it
361-
* means current client does not realize this tracked subscription has gone, and we should forcibly timeout current subscription, and
362-
* schedule a new one.
360+
* If the server sends out check-in message, and there is a active tracked active subscription in client side at the same time,
361+
* it means current client does not realize this tracked subscription has gone, and we should forcibly timeout current
362+
* subscription, and schedule a new one.
363363
*
364364
* This API only works when issuing subscription via SendAutoResubscribeRequest.
365365
*/

src/app/ReadPrepareParams.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ struct ReadPrepareParams
5050
bool mIsPeerLIT = false;
5151

5252
// see ReadClient::OnActiveModeNotification
53-
bool mCatsMatchCheckIn = true;
53+
bool mCatsMatchCheckIn = true;
5454

5555
ReadPrepareParams() {}
5656
ReadPrepareParams(const SessionHandle & sessionHandle) { mSessionHolder.Grab(sessionHandle); }

src/controller/tests/data_model/TestRead.cpp

+5-5
Original file line numberDiff line numberDiff line change
@@ -2355,8 +2355,8 @@ TEST_F(TestRead, TestSubscribe_OnActiveModeNotification)
23552355
}
23562356

23572357
/**
2358-
* When all tracked subscriptions go away in server, check-in message is received and OnActiveModeNotification is called in client side, the tracked
2359-
* subscription would be torn down and a new one would be rescheduled in client side.
2358+
* When all tracked subscriptions go away in server, check-in message is received and OnActiveModeNotification is called in client
2359+
* side, the tracked subscription would be torn down and a new one would be rescheduled in client side.
23602360
*/
23612361
TEST_F(TestRead, TestSubscribe_SubGoAwayInserverOnActiveModeNotification)
23622362
{
@@ -2427,8 +2427,8 @@ TEST_F(TestRead, TestSubscribe_SubGoAwayInserverOnActiveModeNotification)
24272427
}
24282428

24292429
/**
2430-
* When all tracked subscriptions go away in server, check-in message is received and OnActiveModeNotification is called in client side, the
2431-
* untracked subscription would be kept.
2430+
* When all tracked subscriptions go away in server, check-in message is received and OnActiveModeNotification is called in client
2431+
* side, the untracked subscription would be kept.
24322432
*/
24332433
TEST_F(TestRead, TestSubscribe_MismatchedSubGoAwayInserverOnActiveModeNotification)
24342434
{
@@ -2454,7 +2454,7 @@ TEST_F(TestRead, TestSubscribe_MismatchedSubGoAwayInserverOnActiveModeNotificati
24542454
attributePathParams[0].mClusterId = Clusters::UnitTesting::Id;
24552455
attributePathParams[0].mAttributeId = Clusters::UnitTesting::Attributes::Boolean::Id;
24562456
readPrepareParams.mCatsMatchCheckIn = false;
2457-
constexpr uint16_t maxIntervalCeilingSeconds = 1;
2457+
constexpr uint16_t maxIntervalCeilingSeconds = 1;
24582458

24592459
readPrepareParams.mMaxIntervalCeilingSeconds = maxIntervalCeilingSeconds;
24602460
readPrepareParams.mIsPeerLIT = true;

0 commit comments

Comments
 (0)