diff options
author | Sven Gothel <[email protected]> | 2022-11-28 22:01:56 +0100 |
---|---|---|
committer | Sven Gothel <[email protected]> | 2022-11-28 22:01:56 +0100 |
commit | 542369fe13a535b817a42f890f51e1ca09ba552c (patch) | |
tree | b732befc4c69d8b7882d07725ba4f75861182cac /trial/direct_bt/dbt_client_server1x.hpp | |
parent | 9d863d14db95d3c30f0b41b8ce3ed89275934f1d (diff) |
clang-tidy fixes part-3 / 4, adding Check `performance-*`
See also jaulib commit 9160f9a4b74f5e5ae07f715025ea4d3aafdf6ab3
Details:
- Using 'const T&' as method argument type where applicable, i.e. not when required to pass value off-thread.
- Notable: Can also hold a default value!
- Using `const T&` in range-loop where applicable
- Drop std::move for trivial types
- ...
Diffstat (limited to 'trial/direct_bt/dbt_client_server1x.hpp')
-rw-r--r-- | trial/direct_bt/dbt_client_server1x.hpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/trial/direct_bt/dbt_client_server1x.hpp b/trial/direct_bt/dbt_client_server1x.hpp index 4d6b4523..9127d66b 100644 --- a/trial/direct_bt/dbt_client_server1x.hpp +++ b/trial/direct_bt/dbt_client_server1x.hpp @@ -93,8 +93,8 @@ class DBTClientServer1x { void test8x_fullCycle(const std::string& suffix, const int max_connections_per_session, const bool expSuccess, const bool server_client_order, - std::shared_ptr<DBTServerTest> server, const BTSecurityLevel secLevelServer, const ExpectedPairing serverExpPairing, - std::shared_ptr<DBTClientTest> client, const BTSecurityLevel secLevelClient, const ExpectedPairing clientExpPairing) + const std::shared_ptr<DBTServerTest>& server, const BTSecurityLevel secLevelServer, const ExpectedPairing serverExpPairing, + const std::shared_ptr<DBTClientTest>& client, const BTSecurityLevel secLevelClient, const ExpectedPairing clientExpPairing) { (void)secLevelServer; (void)serverExpPairing; @@ -318,7 +318,7 @@ class DBTClientServer1x { DBTEndpoint::stopDiscovery(adapter, true /* current_exp_discovering_state */); } } - const int count = manager->removeChangedAdapterSetCallback(myChangedAdapterSetFunc); - fprintf_td(stderr, "****** EOL Removed ChangedAdapterSetCallback %d\n", count); + const BTManager::size_type count = manager->removeChangedAdapterSetCallback(myChangedAdapterSetFunc); + fprintf_td(stderr, "****** EOL Removed ChangedAdapterSetCallback %zu\n", (size_t)count); } }; |