Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1e2e772b authored by Myles Watson's avatar Myles Watson Committed by Automerger Merge Worker
Browse files

Merge "IWYU: Remove base::StringPrintf from tests" into main am: 697793ec am: e52982ba 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/3110441



Change-Id: I85ea82bf9015a4fc67fec2875f45175905d3ca64
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 249883c4 e52982ba

system/test/fake/fake_looper.cc

+0 −4
Original line number Diff line number Diff line
@@ -16,18 +16,14 @@ 


#include "test/fake/fake_looper.h"



#include <base/strings/stringprintf.h>

#include <bluetooth/log.h>

#include <gtest/gtest.h>

#include <stddef.h>

#include <stdlib.h>



#include <condition_variable>

#include <deque>

#include <mutex>

#include <queue>



#include "os/log.h"

#include "osi/include/allocator.h"

#include "test/fake/fake_thread.h"

system/test/fake/fake_thread.cc

+0 −1
Original line number Diff line number Diff line
@@ -16,7 +16,6 @@ 


#include "test/fake/fake_thread.h"



#include <base/strings/stringprintf.h>

#include <bluetooth/log.h>

#include <gtest/gtest.h>

#include <stdlib.h>

system/test/headless/interface.h

+16 −17
Original line number Diff line number Diff line
@@ -16,7 +16,6 @@ 


#pragma once



#include <base/strings/stringprintf.h>

#include <bluetooth/log.h>



#include <deque>
@@ -48,6 +47,7 @@ inline std::string callback_text(const Callback& callback) { 
    CASE_RETURN_TEXT(Callback::DiscoveryStateChanged);

    CASE_RETURN_TEXT(Callback::RemoteDeviceProperties);

  }

  RETURN_UNKNOWN_TYPE_STRING(Callback, callback);

}



struct callback_data_t {
@@ -144,15 +144,15 @@ struct acl_state_changed_params_t : public callback_params_t { 
  uint16_t acl_handle;



  std::string ToString() const override {

    return base::StringPrintf(

        "status:%s remote_bd_addr:%s state:%s transport:%s reason:%s"

        " direction:%s handle:%hu",

        bt_status_text(status).c_str(), remote_bd_addr.ToString().c_str(),

    return fmt::format(

        "status:{} remote_bd_addr:{} state:{} transport:{} reason:{}"

        " direction:{} handle:{}",

        bt_status_text(status), remote_bd_addr.ToString(),

        (state == BT_ACL_STATE_CONNECTED) ? "CONNECTED" : "DISCONNECTED",

        bt_transport_text(static_cast<const tBT_TRANSPORT>(transport_link_type))

            .c_str(),

        bt_status_text(static_cast<const bt_status_t>(hci_reason)).c_str(),

        bt_conn_direction_text(direction).c_str(), acl_handle);

        bt_transport_text(

            static_cast<const tBT_TRANSPORT>(transport_link_type)),

        bt_status_text(static_cast<const bt_status_t>(hci_reason)),

        bt_conn_direction_text(direction), acl_handle);

  }

};
@@ -168,8 +168,7 @@ struct discovery_state_changed_params_t : public callback_params_t { 


  bt_discovery_state_t state;

  std::string ToString() const override {

    return base::StringPrintf("state:%s",

                              bt_discovery_state_text(state).c_str());

    return fmt::format("state:{}", bt_discovery_state_text(state));

  }

};
@@ -187,8 +186,8 @@ struct adapter_properties_params_t : public callback_params_with_properties_t { 
  bt_status_t status;



  std::string ToString() const override {

    return base::StringPrintf("status:%s num_properties:%zu",

                              bt_status_text(status).c_str(), num_properties());

    return fmt::format("status:{} num_properties:{}", bt_status_text(status),

                       num_properties());

  }

};
@@ -210,9 +209,9 @@ struct remote_device_properties_params_t 
  RawAddress bd_addr;



  std::string ToString() const override {

    return base::StringPrintf("status:%s bd_addr:%s num_properties:%zu",

                              bt_status_text(status).c_str(),

                              bd_addr.ToString().c_str(), num_properties());

    return fmt::format("status:{} bd_addr:{} num_properties:{}",

                       bt_status_text(status), bd_addr.ToString(),

                       num_properties());

  }

};
@@ -225,7 +224,7 @@ struct device_found_params_t : public callback_params_with_properties_t { 
  virtual ~device_found_params_t() {}



  std::string ToString() const override {

    return base::StringPrintf("num_properties:%zu", num_properties());

    return fmt::format("num_properties:{}", num_properties());

  }

};

system/test/headless/stopwatch.h

+3 −4
Original line number Diff line number Diff line
@@ -16,7 +16,7 @@ 


#pragma once



#include <base/strings/stringprintf.h>

#include <bluetooth/log.h>



#include <chrono>

#include <cstdint>
@@ -40,9 +40,8 @@ class Stopwatch { 
  std::string ToString() { return ToString(""); }



  std::string ToString(const std::string& comment) {

    return base::StringPrintf("%s: %lu ms %s", name_.c_str(),

                              static_cast<unsigned long>(LapMs()),

                              comment.c_str());

    return fmt::format("{}: {} ms {}", name_,

                       static_cast<unsigned long>(LapMs()), comment);

  }



 private:

system/test/headless/utils/power_mode_client.h

+3 −5
Original line number Diff line number Diff line
@@ -18,7 +18,6 @@ 


#define LOG_TAG "bt_headless_mode"



#include <base/strings/stringprintf.h>

#include <bluetooth/log.h>



#include <future>
@@ -67,10 +66,9 @@ struct power_mode_callback_t { 
  tHCI_STATUS hci_status;



  std::string ToString() const {

    return base::StringPrintf("bd_addr:%s pm_status:%s value:%hu hci_status:%s",

                              bd_addr.ToString().c_str(),

                              power_mode_status_text(status).c_str(), value,

                              hci_status_code_text(hci_status).c_str());

    return fmt::format("bd_addr:{} pm_status:{} value:{} hci_status:{}",

                       bd_addr.ToString(), power_mode_status_text(status),

                       value, hci_status_code_text(hci_status));

  }

};