braces always on newline

Author: paul-szczepanek-arm

connectivity/FEATURE_BLE/include/ble/common/ble/BLETypes.h

@@ -311,7 +311,8 @@ class PasskeyAscii {
  * Returns true if every byte is equal to zero
  */
 template <class byte_array_class>
-bool is_all_zeros(byte_array_class &byte_array) {
+bool is_all_zeros(byte_array_class &byte_array)
+{
     for (size_t i = 0; i < byte_array.size(); i++) {
         if (byte_array[i] != 0) {
             return false;
@@ -324,7 +325,8 @@ bool is_all_zeros(byte_array_class &byte_array) {
  * Zero out all bytes
  */
 template <class byte_array_class>
-void set_all_zeros(byte_array_class &byte_array) {
+void set_all_zeros(byte_array_class &byte_array)
+{
     memset(&byte_array[0], 0x00, byte_array.size());
 }
 

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/TESTS/cordio_hci/driver/main.cpp

@@ -49,12 +49,14 @@ enum initialization_state_t {
 
 static initialization_state_t initialization_state = WAITING_FOR_INITIALIZATION;
 
-static void process_ble_events(BLE::OnEventsToProcessCallbackContext* context) {
+static void process_ble_events(BLE::OnEventsToProcessCallbackContext* context)
+{
     BLE &ble = BLE::Instance();
     event_queue.call(callback(&ble, &BLE::processEvents));
 }
 
-static void on_initialization_complete(BLE::InitializationCompleteCallbackContext *params) {
+static void on_initialization_complete(BLE::InitializationCompleteCallbackContext *params)
+{
     if (params->error == BLE_ERROR_NONE) {
         initialization_state = INITIALIZATION_SUCCESS;
     } else {
@@ -64,7 +66,8 @@ static void on_initialization_complete(BLE::InitializationCompleteCallbackContex
     event_queue.break_dispatch();
 }
 
-static void test_stack_initialization() {
+static void test_stack_initialization()
+{
     BLE &ble = BLE::Instance();
     ble.onEventsToProcess(process_ble_events);
     ble.init(on_initialization_complete);
@@ -102,14 +105,16 @@ Case cases[] = {
     Case("Test cordio stack reset sequence", test_stack_initialization),
 };
 
-utest::v1::status_t greentea_test_setup(const size_t number_of_cases) {
+utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
+{
     GREENTEA_SETUP(15, "default_auto");
     return verbose_test_setup_handler(number_of_cases);
 }
 
 Specification specification(greentea_test_setup, cases, greentea_test_teardown_handler);
 
-int main() {
+int main()
+{
     return !Harness::run(specification);
 }
 

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/doc/PortingGuide.md

@@ -233,7 +233,8 @@ At runtime, you can get stack parameters from the controller:
 **Example:**
 
 ```
-void HCIDriver::handle_reset_sequence(uint8_t *pMsg) {
+void HCIDriver::handle_reset_sequence(uint8_t *pMsg)
+{
     // only accept command complete event:
     if (*pMsg != HCI_CMD_CMPL_EVT) {
         return;
@@ -440,7 +441,8 @@ ble::CordioHCIDriver& ble_cordio_get_hci_driver();
 **Example:**
 
 ```
-ble::CordioHCIDriver& ble_cordio_get_hci_driver() {
+ble::CordioHCIDriver& ble_cordio_get_hci_driver()
+{
     static ble::vendor::target_name::TransportDriver transport_driver(
         /* transport parameters */
     );

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/driver/CordioHCIDriver.cpp

@@ -286,7 +286,8 @@ uint16_t CordioHCIDriver::write(uint8_t type, uint16_t len, uint8_t *pData)
 void CordioHCIDriver::on_host_stack_inactivity()
 {
 }
-void CordioHCIDriver::handle_test_end(bool success, uint16_t packets) {
+void CordioHCIDriver::handle_test_end(bool success, uint16_t packets)
+{
     if (_test_end_handler) {
         _test_end_handler(success, packets);
         _test_end_handler = nullptr;
@@ -364,7 +365,8 @@ ble_error_t CordioHCIDriver::rf_test_end()
     return BLE_ERROR_NO_MEM;
 }
 
-ble_error_t CordioHCIDriver::set_tx_power(int8_t level_db) {
+ble_error_t CordioHCIDriver::set_tx_power(int8_t level_db)
+{
     return BLE_ERROR_NOT_IMPLEMENTED;
 }
 

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/source/PalAttClientImpl.cpp

@@ -35,7 +35,8 @@ PalAttClient::~PalAttClient() {}
 /**
 * @see ble::PalAttClient::exchange_mtu_request
 */
-ble_error_t PalAttClient::exchange_mtu_request(connection_handle_t connection) {
+ble_error_t PalAttClient::exchange_mtu_request(connection_handle_t connection)
+{
     AttcMtuReq(connection, pAttCfg->mtu);
     return BLE_ERROR_NONE;
 }
@@ -272,19 +273,22 @@ ble_error_t PalAttClient::execute_write_request(
 /**
 * @see ble::PalAttClient::initialize
 */
-ble_error_t PalAttClient::initialize() {
+ble_error_t PalAttClient::initialize()
+{
     return BLE_ERROR_NONE;
 }
 
 /**
 * @see ble::PalAttClient::terminate
 */
-ble_error_t PalAttClient::terminate() {
+ble_error_t PalAttClient::terminate()
+{
     return BLE_ERROR_NONE;
 }
 
 // singleton of the ARM Cordio client
-PalAttClient &PalAttClient::get_client() {
+PalAttClient &PalAttClient::get_client()
+{
     static PalAttClient _client;
     return _client;
 }
@@ -302,22 +306,25 @@ void PalAttClient::when_transaction_timeout(
 }
 
 // convert an array of byte to an uint16_t
-uint16_t PalAttClient::to_uint16_t(const uint8_t *array) {
+uint16_t PalAttClient::to_uint16_t(const uint8_t *array)
+{
     uint16_t result;
     memcpy(&result, array, sizeof(result));
     return result;
 }
 
 template<typename T>
-bool PalAttClient::event_handler(const attEvt_t *event) {
+bool PalAttClient::event_handler(const attEvt_t *event)
+{
     if (T::can_convert(event)) {
         generated_handler(event, T::convert);
         return true;
     }
     return false;
 }
 
-bool PalAttClient::timeout_event_handler(const attEvt_t *event) {
+bool PalAttClient::timeout_event_handler(const attEvt_t *event)
+{
     if (event->hdr.status != ATT_ERR_TIMEOUT) {
         return false;
     }
@@ -362,7 +369,8 @@ void PalAttClient::on_transaction_timeout(
     }
 }
 
-void PalAttClient::att_client_handler(const attEvt_t *event) {
+void PalAttClient::att_client_handler(const attEvt_t *event)
+{
 #if BLE_FEATURE_GATT_CLIENT
     if (event->hdr.status == ATT_SUCCESS && event->hdr.event == ATT_MTU_UPDATE_IND) {
         ble::BLEInstanceBase &ble = ble::BLEInstanceBase::deviceInstance();

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/source/PalGattClientImpl.cpp

@@ -34,7 +34,8 @@ PalGattClient::PalGattClient(PalAttClient& client) : _event_handler(nullptr), _c
 }
 
 
-ble_error_t PalGattClient::exchange_mtu(connection_handle_t connection) {
+ble_error_t PalGattClient::exchange_mtu(connection_handle_t connection)
+{
     return _client.exchange_mtu_request(connection);
 }
 
@@ -221,12 +222,14 @@ ble_error_t PalGattClient::execute_write_queue(
 }
 
 
-ble_error_t PalGattClient::initialize() {
+ble_error_t PalGattClient::initialize()
+{
     return _client.initialize();
 }
 
 
-ble_error_t PalGattClient::terminate() {
+ble_error_t PalGattClient::terminate()
+{
     return _client.initialize();
 }
 

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/source/PalGenericAccessServiceImpl.cpp

@@ -22,7 +22,8 @@ namespace ble {
 
 #if 0 // Disabled until reworked and reintroduced to GattServer API
 
-virtual ble_error_t PalGenericAccessService::get_device_name_length(uint8_t& length) {
+virtual ble_error_t PalGenericAccessService::get_device_name_length(uint8_t& length)
+{
 #if BLE_FEATURE_GATT_SERVER
         const uint8_t* name = nullptr;
         uint16_t actual_length = 0;
@@ -112,7 +113,8 @@ ble_error_t PalGenericAccessService::set_peripheral_preferred_connection_paramet
 
 
 #if BLE_FEATURE_GATT_SERVER
-ble::GattServer& PalGenericAccessService::gatt_server() {
+ble::GattServer& PalGenericAccessService::gatt_server()
+{
     return ble::GattServer::getInstance();
 }
 

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/source/PalSecurityManagerImpl.cpp

@@ -550,7 +550,8 @@ ble_error_t PalSecurityManager::send_keypress_notification(
 
 
 
-ble_error_t PalSecurityManager::generate_secure_connections_oob() {
+ble_error_t PalSecurityManager::generate_secure_connections_oob()
+{
     uint8_t oobLocalRandom[SMP_RAND_LEN];
     SecRand(oobLocalRandom, SMP_RAND_LEN);
     DmSecCalcOobReq(oobLocalRandom, _public_key_x);
@@ -586,7 +587,8 @@ PalSecurityManager& PalSecurityManager::get_security_manager()
 }
 
 
-bool PalSecurityManager::sm_handler(const wsfMsgHdr_t* msg) {
+bool PalSecurityManager::sm_handler(const wsfMsgHdr_t* msg)
+{
     PalSecurityManager& self = get_security_manager();
     PalSecurityManagerEventHandler* handler = self.get_event_handler();
 
@@ -957,7 +959,8 @@ void PalSecurityManager::queue_remove_device_from_resolving_list(
 }
 
 
-void PalSecurityManager::queue_clear_resolving_list() {
+void PalSecurityManager::queue_clear_resolving_list()
+{
     // Remove any pending control blocks, there's no point executing them as we're about to queue the list
     clear_privacy_control_blocks();
 
@@ -973,7 +976,8 @@ void PalSecurityManager::queue_clear_resolving_list() {
 }
 
 
-void PalSecurityManager::clear_privacy_control_blocks() {
+void PalSecurityManager::clear_privacy_control_blocks()
+{
     while(_pending_privacy_control_blocks != nullptr)
     {
         PrivacyControlBlock* next = _pending_privacy_control_blocks->next();
@@ -1026,7 +1030,8 @@ void PalSecurityManager::process_privacy_control_blocks(bool cb_completed)
 }
 
 
-void PalSecurityManager::cleanup_peer_csrks() {
+void PalSecurityManager::cleanup_peer_csrks()
+{
     for (size_t i = 0; i < DM_CONN_MAX; ++i) {
         if (_peer_csrks[i]) {
             delete _peer_csrks[i];
@@ -1040,7 +1045,8 @@ void PalSecurityManager::set_event_handler(PalSecurityManagerEventHandler *event
     _pal_event_handler = event_handler;
 }
 
-PalSecurityManagerEventHandler* PalSecurityManager::get_event_handler() {
+PalSecurityManagerEventHandler* PalSecurityManager::get_event_handler()
+{
     return _pal_event_handler;
 }
 

connectivity/FEATURE_BLE/libraries/TARGET_CORDIO/source/PalSigningMonitorImpl.cpp

@@ -22,7 +22,8 @@
 
 namespace ble {
 
-void PalSigningMonitor::set_signing_event_handler(SecurityManager *handler) {
+void PalSigningMonitor::set_signing_event_handler(SecurityManager *handler)
+{
 #if BLE_FEATURE_GATT_CLIENT
     BLEInstanceBase::deviceInstance().getGattClient().set_signing_event_handler(handler);
 #endif // BLE_FEATURE_GATT_CLIENT

connectivity/FEATURE_BLE/libraries/ble-api-implementation/source/GapImpl.cpp

@@ -2730,7 +2730,8 @@ void Gap::prepare_legacy_advertising_set()
     _existing_sets.set(LEGACY_ADVERTISING_HANDLE);
 }
 
-void Gap::setEventHandler(Gap::EventHandler *handler) {
+void Gap::setEventHandler(Gap::EventHandler *handler)
+{
     _event_handler = handler;
 }