diff options
author | Ajay Panicker <apanicke@google.com> | 2018-09-10 12:01:50 -0700 |
---|---|---|
committer | Ajay Panicker <apanicke@google.com> | 2018-09-11 17:35:27 +0000 |
commit | 746da53d10e57322f775b0d2cc68a0e167002fb7 (patch) | |
tree | c51fd531ec7ee74f6ab844d75a193464eba40291 /packet/avrcp | |
parent | a81834c8260113c8e19de5c75a5f5147dd9e5e33 (diff) | |
download | bt-746da53d10e57322f775b0d2cc68a0e167002fb7.tar.gz |
Add extractBE to the Bluetooth Packets Iterators
ExtractBE allows data to be extracted from Bluetooth packets in a big
endian format.
Bug: 114751344
Test: run net_test_btpackets
Change-Id: Ib01d726893e106d34c11048acc09fee201d81c09
Diffstat (limited to 'packet/avrcp')
-rw-r--r-- | packet/avrcp/avrcp_browse_packet.cc | 4 | ||||
-rw-r--r-- | packet/avrcp/avrcp_packet.h | 2 | ||||
-rw-r--r-- | packet/avrcp/change_path.cc | 6 | ||||
-rw-r--r-- | packet/avrcp/get_element_attributes_packet.cc | 4 | ||||
-rw-r--r-- | packet/avrcp/get_folder_items.cc | 8 | ||||
-rw-r--r-- | packet/avrcp/get_item_attributes.cc | 8 | ||||
-rw-r--r-- | packet/avrcp/play_item.cc | 6 | ||||
-rw-r--r-- | packet/avrcp/register_notification_packet.cc | 4 | ||||
-rw-r--r-- | packet/avrcp/set_addressed_player.cc | 4 | ||||
-rw-r--r-- | packet/avrcp/set_browsed_player.cc | 4 | ||||
-rw-r--r-- | packet/avrcp/vendor_packet.cc | 4 |
11 files changed, 27 insertions, 27 deletions
diff --git a/packet/avrcp/avrcp_browse_packet.cc b/packet/avrcp/avrcp_browse_packet.cc index f4a769151..07b548599 100644 --- a/packet/avrcp/avrcp_browse_packet.cc +++ b/packet/avrcp/avrcp_browse_packet.cc @@ -61,7 +61,7 @@ BrowsePdu BrowsePacket::GetPdu() const { uint16_t BrowsePacket::GetLength() const { auto it = begin() + static_cast<size_t>(1); - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } bool BrowsePacket::IsValid() const { @@ -88,4 +88,4 @@ std::pair<size_t, size_t> BrowsePacket::GetPayloadIndecies() const { } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/avrcp_packet.h b/packet/avrcp/avrcp_packet.h index fc28f2f8a..ee0e0d5ad 100644 --- a/packet/avrcp/avrcp_packet.h +++ b/packet/avrcp/avrcp_packet.h @@ -108,4 +108,4 @@ class Packet : public ::bluetooth::Packet { }; } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/change_path.cc b/packet/avrcp/change_path.cc index 3c69903bf..b5f9ab4ac 100644 --- a/packet/avrcp/change_path.cc +++ b/packet/avrcp/change_path.cc @@ -52,7 +52,7 @@ bool ChangePathResponseBuilder::Serialize( uint16_t ChangePathRequest::GetUidCounter() const { auto it = begin() + BrowsePacket::kMinSize(); - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } Direction ChangePathRequest::GetDirection() const { @@ -62,7 +62,7 @@ Direction ChangePathRequest::GetDirection() const { uint64_t ChangePathRequest::GetUid() const { auto it = begin() + BrowsePacket::kMinSize() + static_cast<size_t>(3); - return base::ByteSwap(it.extract<uint64_t>()); + return it.extractBE<uint64_t>(); } bool ChangePathRequest::IsValid() const { @@ -109,4 +109,4 @@ bool ChangePathRequestBuilder::Serialize( } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/get_element_attributes_packet.cc b/packet/avrcp/get_element_attributes_packet.cc index c26207e61..00ed59694 100644 --- a/packet/avrcp/get_element_attributes_packet.cc +++ b/packet/avrcp/get_element_attributes_packet.cc @@ -40,7 +40,7 @@ std::vector<Attribute> GetElementAttributesRequest::GetAttributesRequested() std::vector<Attribute> attribute_list; for (size_t i = 0; i < number_of_attributes; i++) { - attribute_list.push_back((Attribute)base::ByteSwap(it.extract<uint32_t>())); + attribute_list.push_back((Attribute)it.extractBE<uint32_t>()); } return attribute_list; @@ -140,4 +140,4 @@ bool GetElementAttributesResponseBuilder::Serialize( } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/get_folder_items.cc b/packet/avrcp/get_folder_items.cc index c95bb4a9c..4e6e3a172 100644 --- a/packet/avrcp/get_folder_items.cc +++ b/packet/avrcp/get_folder_items.cc @@ -236,12 +236,12 @@ Scope GetFolderItemsRequest::GetScope() const { uint32_t GetFolderItemsRequest::GetStartItem() const { auto it = begin() + BrowsePacket::kMinSize() + static_cast<size_t>(1); - return base::ByteSwap(it.extract<uint32_t>()); + return it.extractBE<uint32_t>(); } uint32_t GetFolderItemsRequest::GetEndItem() const { auto it = begin() + BrowsePacket::kMinSize() + static_cast<size_t>(5); - return base::ByteSwap(it.extract<uint32_t>()); + return it.extractBE<uint32_t>(); } uint8_t GetFolderItemsRequest::GetNumAttributes() const { @@ -263,7 +263,7 @@ std::vector<Attribute> GetFolderItemsRequest::GetAttributesRequested() const { // to have this function return a vector with all the attributes for (size_t i = 0; i < number_of_attributes; i++) { - attribute_list.push_back((Attribute)base::ByteSwap(it.extract<uint32_t>())); + attribute_list.push_back((Attribute)it.extractBE<uint32_t>()); } return attribute_list; @@ -342,4 +342,4 @@ bool GetFolderItemsRequestBuilder::Serialize( } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/get_item_attributes.cc b/packet/avrcp/get_item_attributes.cc index 34304744c..145dfed7b 100644 --- a/packet/avrcp/get_item_attributes.cc +++ b/packet/avrcp/get_item_attributes.cc @@ -91,12 +91,12 @@ Scope GetItemAttributesRequest::GetScope() const { uint64_t GetItemAttributesRequest::GetUid() const { auto it = begin() + BrowsePacket::kMinSize() + static_cast<size_t>(1); - return base::ByteSwap(it.extract<uint64_t>()); + return it.extractBE<uint64_t>(); } uint16_t GetItemAttributesRequest::GetUidCounter() const { auto it = begin() + BrowsePacket::kMinSize() + static_cast<size_t>(9); - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } uint8_t GetItemAttributesRequest::GetNumAttributes() const { @@ -111,7 +111,7 @@ std::vector<Attribute> GetItemAttributesRequest::GetAttributesRequested() std::vector<Attribute> attribute_list; for (size_t i = 0; i < number_of_attributes; i++) { - attribute_list.push_back((Attribute)base::ByteSwap(it.extract<uint32_t>())); + attribute_list.push_back((Attribute)it.extractBE<uint32_t>()); } return attribute_list; @@ -147,4 +147,4 @@ std::string GetItemAttributesRequest::ToString() const { } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/play_item.cc b/packet/avrcp/play_item.cc index 27b242bd1..d7bb9b73d 100644 --- a/packet/avrcp/play_item.cc +++ b/packet/avrcp/play_item.cc @@ -53,12 +53,12 @@ Scope PlayItemRequest::GetScope() const { uint64_t PlayItemRequest::GetUid() const { auto it = begin() + VendorPacket::kMinSize() + static_cast<size_t>(1); - return base::ByteSwap(it.extract<uint64_t>()); + return it.extractBE<uint64_t>(); } uint16_t PlayItemRequest::GetUidCounter() const { auto it = begin() + VendorPacket::kMinSize() + static_cast<size_t>(9); - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } bool PlayItemRequest::IsValid() const { @@ -86,4 +86,4 @@ std::string PlayItemRequest::ToString() const { } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/register_notification_packet.cc b/packet/avrcp/register_notification_packet.cc index 86115e154..adb5e59c0 100644 --- a/packet/avrcp/register_notification_packet.cc +++ b/packet/avrcp/register_notification_packet.cc @@ -234,7 +234,7 @@ Event RegisterNotificationRequest::GetEventRegistered() const { uint32_t RegisterNotificationRequest::GetInterval() const { auto it = begin() + VendorPacket::kMinSize() + static_cast<size_t>(1); - return base::ByteSwap(it.extract<uint32_t>()); + return it.extractBE<uint32_t>(); } bool RegisterNotificationRequest::IsValid() const { @@ -288,4 +288,4 @@ bool RegisterNotificationRequestBuilder::Serialize( } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/set_addressed_player.cc b/packet/avrcp/set_addressed_player.cc index f4b4dddb5..d1a990925 100644 --- a/packet/avrcp/set_addressed_player.cc +++ b/packet/avrcp/set_addressed_player.cc @@ -48,7 +48,7 @@ bool SetAddressedPlayerResponseBuilder::Serialize( uint16_t SetAddressedPlayerRequest::GetPlayerId() const { auto it = begin() + VendorPacket::kMinSize(); - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } bool SetAddressedPlayerRequest::IsValid() const { @@ -74,4 +74,4 @@ std::string SetAddressedPlayerRequest::ToString() const { } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/set_browsed_player.cc b/packet/avrcp/set_browsed_player.cc index b8463fa6f..fa0c76842 100644 --- a/packet/avrcp/set_browsed_player.cc +++ b/packet/avrcp/set_browsed_player.cc @@ -80,7 +80,7 @@ bool SetBrowsedPlayerResponseBuilder::Serialize( uint16_t SetBrowsedPlayerRequest::GetPlayerId() const { auto it = begin() + BrowsePacket::kMinSize(); - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } bool SetBrowsedPlayerRequest::IsValid() const { @@ -100,4 +100,4 @@ std::string SetBrowsedPlayerRequest::ToString() const { } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth diff --git a/packet/avrcp/vendor_packet.cc b/packet/avrcp/vendor_packet.cc index 982cf8176..4525cfbd9 100644 --- a/packet/avrcp/vendor_packet.cc +++ b/packet/avrcp/vendor_packet.cc @@ -96,7 +96,7 @@ PacketType VendorPacket::GetPacketType() const { uint16_t VendorPacket::GetParameterLength() const { auto it = begin() + Packet::kMinSize() + static_cast<size_t>(5); // Swap to little endian - return base::ByteSwap(it.extract<uint16_t>()); + return it.extractBE<uint16_t>(); } bool VendorPacket::IsValid() const { @@ -128,4 +128,4 @@ std::string VendorPacket::ToString() const { } } // namespace avrcp -} // namespace bluetooth
\ No newline at end of file +} // namespace bluetooth |