[PATCH 1/7] android/client: Omit length parameter for hex string data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



There is no need to pass length parameter to functions. Length of hex
strings is automatically calculated from string in GET_VERIFY_HEX_STRING
macro.
---
 android/client/if-gatt.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/android/client/if-gatt.c b/android/client/if-gatt.c
index 2b874c3..112a49d 100644
--- a/android/client/if-gatt.c
+++ b/android/client/if-gatt.c
@@ -1729,10 +1729,8 @@ static void scan_filter_add_remove_p(int argc, const char **argv)
 	VERIFY_UUID(9, &p_uuid_mask);
 	VERIFY_ADDR_ARG(10, &bd_addr);
 	VERIFY_ADDR_TYPE(11, addr_type);
-	VERIFY_DATA_LEN(12, data_len);
-	GET_VERIFY_HEX_STRING(13, p_data, data_len);
-	VERIFY_MASK_LEN(14, mask_len);
-	GET_VERIFY_HEX_STRING(15, p_mask, mask_len);
+	GET_VERIFY_HEX_STRING(12, p_data, data_len);
+	GET_VERIFY_HEX_STRING(13, p_mask, mask_len);
 
 	EXEC(if_gatt->client->scan_filter_add_remove, client_if, action,
 		filt_type, filt_index, company_id, company_id_mask,
@@ -1862,7 +1860,7 @@ static struct method client_methods[] = {
 	STD_METHODCH(scan_filter_add_remove, "<client_if> <action> <filt_type>"
 			" <filt_index> <company_id> <company_id_mask>"
 			" [<p_uuid>] <p_uuid_mask> <addr> <addr_type>"
-			" <data_len> [<p_data>] <mask_len> [<p_mask>]"),
+			" [<p_data>] [<p_mask>]"),
 	STD_METHODCH(scan_filter_clear, "<client_if> <filt_index>"),
 	STD_METHODCH(scan_filter_enable, "<client_if> [<enable>]"),
 #else
-- 
1.9.3

--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux