control: use braces with conditions and logging macros
Removes warnings and may prevent future clumsy mistakes. Signed-off-by: Thomas Tsou <ttsou@vt.edu>
This commit is contained in:
parent
215b8f7e5d
commit
f6ed307167
|
@ -361,8 +361,11 @@ bool callManagementDispatchGSM(TransactionEntry& transaction, LogicalChannel* LC
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (message) LOG(NOTICE) << "no support for message " << *message << " from " << transaction.subscriber();
|
if (message) {
|
||||||
else LOG(NOTICE) << "no support for unrecognized message from " << transaction.subscriber();
|
LOG(NOTICE) << "no support for message " << *message << " from " << transaction.subscriber();
|
||||||
|
} else {
|
||||||
|
LOG(NOTICE) << "no support for unrecognized message from " << transaction.subscriber();
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
// If we got here, we're ignoring the message.
|
// If we got here, we're ignoring the message.
|
||||||
|
|
|
@ -242,8 +242,11 @@ void Control::LocationUpdatingController(const L3LocationUpdatingRequest* lur, S
|
||||||
// Otherwise, we are here because of open registration.
|
// Otherwise, we are here because of open registration.
|
||||||
// Either way, we're going to register a phone if we arrive here.
|
// Either way, we're going to register a phone if we arrive here.
|
||||||
|
|
||||||
if (success) LOG(INFO) << "registration SUCCESS: " << mobID;
|
if (success) {
|
||||||
else LOG(INFO) << "registration ALLOWED: " << mobID;
|
LOG(INFO) << "registration SUCCESS: " << mobID;
|
||||||
|
} else {
|
||||||
|
LOG(INFO) << "registration ALLOWED: " << mobID;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
// Send the "short name".
|
// Send the "short name".
|
||||||
|
|
Reference in New Issue