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

Commit 03e5fac5 authored by Henri Chataing's avatar Henri Chataing Committed by Gerrit Code Review
Browse files

Merge "RootCanal: Inline gd_defaults and fluoride_common_options in Android.bp"

parents 00b51e33 47073f88

tools/rootcanal/Android.bp

+17 −9
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ cc_defaults { 
    tidy_checks: [

        "-*",

        "readability-*",



        "-readability-function-size",

        "-readability-identifier-length",

        "-readability-implicit-bool-conversion",
@@ -32,12 +31,22 @@ cc_defaults { 
    tidy_flags: [

        "--header-filter=^.*tools\\/rootcanal\\/(model|include|net|desktop)\\/(.(?!\\.pb\\.h))*$",

    ],

    defaults: [

        "fluoride_common_options",

        "gd_defaults",

    ],

    sanitize: {

        address: true,

        all_undefined: true,

        misc_undefined: ["bounds"],

    },

    c_std: "c99",

    cpp_std: "c++17",

    cflags: [

        "-Wall",

        "-Werror",

        "-Wextra",

        "-fvisibility=hidden",

        "-DGOOGLE_PROTOBUF_NO_RTTI",

    ],

    include_dirs: [

        "packages/modules/Bluetooth/system/gd",

    ],

    local_include_dirs: [

        "include",
@@ -148,6 +157,9 @@ cc_library_host_shared { 
    defaults: [

        "rootcanal_defaults",

    ],

    sanitize: {

        address: false,

    },

    srcs: [

        ":BluetoothPacketSources",

        ":lib_sources",
@@ -395,10 +407,6 @@ cc_binary_host { 
        "libprotobuf-cpp-full",

        "libscriptedbeaconpayload-protos-lite",

    ],

    sanitize: {

        address: true,

        all_undefined: true,

    },

    target: {

        // TODO(b/181290178) remove it when sanitize option is supported by linux_bionic as well

        linux_bionic: {

tools/rootcanal/model/controller/dual_mode_controller.cc

+1 −1
Original line number Diff line number Diff line
@@ -2949,7 +2949,7 @@ void DualModeController::CsrReadVarid(CsrVarid varid, 
}



void DualModeController::CsrWriteVarid(

    CsrVarid varid, std::vector<uint8_t> const& value) const {

    CsrVarid varid, std::vector<uint8_t> const& /*value*/) const {

  INFO(id_, "Unsupported write of CSR varid 0x{:04x}", varid);

}

tools/rootcanal/model/controller/le_advertiser.cc

+1 −1
Original line number Diff line number Diff line
@@ -1594,7 +1594,7 @@ ErrorCode LinkLayerController::LeSetPeriodicAdvertisingEnable( 
}



uint16_t ExtendedAdvertiser::GetMaxPeriodicAdvertisingDataLength(

    slots periodic_advertising_interval) {

    slots /*periodic_advertising_interval*/) {

  // TODO: evaluate the maximum length of the advertising PDU that can

  // be physically sent in the advertising interval.

  return max_extended_advertising_pdu_size;

tools/rootcanal/model/devices/device.h

+2 −2
Original line number Diff line number Diff line
@@ -53,8 +53,8 @@ class Device { 
  virtual void Close();



  virtual void ReceiveLinkLayerPacket(

      model::packets::LinkLayerPacketView packet, Phy::Type type,

      int8_t rssi){};

      model::packets::LinkLayerPacketView /*packet*/, Phy::Type /*type*/,

      int8_t /*rssi*/){};



  void SendLinkLayerPacket(

      std::shared_ptr<model::packets::LinkLayerPacketBuilder> packet,

tools/rootcanal/model/hci/hci_sniffer.h

+1 −1
Original line number Diff line number Diff line
@@ -44,7 +44,7 @@ class HciSniffer : public HciTransport { 
  static std::shared_ptr<HciTransport> Create(

      std::shared_ptr<HciTransport> transport,

      std::shared_ptr<std::ostream> outputStream = nullptr,

      std::shared_ptr<PcapFilter> filter = nullptr) {

      std::shared_ptr<PcapFilter> /*filter*/ = nullptr) {

    return std::make_shared<HciSniffer>(transport, outputStream);

  }