commit 08dda210252061331fb948f5a7d49d699ee9a5e2
author Josh Gao <jmgao@google.com> Mon Sep 26 21:18:58 2016 -0700
committer Josh Gao <jmgao@google.com> Mon Oct 10 13:44:00 2016 -0700
tree 270d578ea9678640de7ff680ca95a889b933f10b
parent b4e8ba2768d060465ad4fe9f0d39c809b08b93d7

Remove useless arguments from is_adb_interface.

Test: mma
Change-Id: I8b7b411d7d5ec9d401b61ed8f817b98c61114d4f

adb.h

diff --git a/adb.h b/adb.h
index 0b9fe5b..b3f0e86 100644
--- a/adb.h
+++ b/adb.h
@@ -199,7 +199,7 @@
 
 // USB device detection.
 #if ADB_HOST
-int is_adb_interface(int vid, int pid, int usb_class, int usb_subclass, int usb_protocol);
+int is_adb_interface(int usb_class, int usb_subclass, int usb_protocol);
 #endif
 
 ConnectionState connection_state(atransport *t);

transport_usb.cpp

diff --git a/transport_usb.cpp b/transport_usb.cpp
index d05d928..d054601 100644
--- a/transport_usb.cpp
+++ b/transport_usb.cpp
@@ -94,7 +94,7 @@
 }
 
 #if ADB_HOST
-int is_adb_interface(int vid, int pid, int usb_class, int usb_subclass, int usb_protocol)
+int is_adb_interface(int usb_class, int usb_subclass, int usb_protocol)
 {
     return (usb_class == ADB_CLASS && usb_subclass == ADB_SUBCLASS && usb_protocol == ADB_PROTOCOL);
 }

usb_linux.cpp

diff --git a/usb_linux.cpp b/usb_linux.cpp
index 500898a..3e5028d 100644
--- a/usb_linux.cpp
+++ b/usb_linux.cpp
@@ -205,9 +205,8 @@
                          interface->bInterfaceProtocol, interface->bNumEndpoints);
 
                     if (interface->bNumEndpoints == 2 &&
-                            is_adb_interface(vid, pid, interface->bInterfaceClass,
-                            interface->bInterfaceSubClass, interface->bInterfaceProtocol))  {
-
+                        is_adb_interface(interface->bInterfaceClass, interface->bInterfaceSubClass,
+                                         interface->bInterfaceProtocol)) {
                         struct stat st;
                         char pathbuf[128];
                         char link[256];

usb_osx.cpp

diff --git a/usb_osx.cpp b/usb_osx.cpp
index adcbb3e..2ee2aae 100644
--- a/usb_osx.cpp
+++ b/usb_osx.cpp
@@ -345,7 +345,7 @@
 
     //* check to make sure interface class, subclass and protocol match ADB
     //* avoid opening mass storage endpoints
-    if (!is_adb_interface(vendor, product, interfaceClass, interfaceSubClass, interfaceProtocol)) {
+    if (!is_adb_interface(interfaceClass, interfaceSubClass, interfaceProtocol)) {
         goto err_bad_adb_interface;
     }
 

usb_windows.cpp

diff --git a/usb_windows.cpp b/usb_windows.cpp
index 4649454..755f07e 100644
--- a/usb_windows.cpp
+++ b/usb_windows.cpp
@@ -553,10 +553,9 @@
     return 0;
   }
 
-  if (is_adb_interface(device_desc.idVendor, device_desc.idProduct,
-      interf_desc.bInterfaceClass, interf_desc.bInterfaceSubClass, interf_desc.bInterfaceProtocol)) {
-
-    if(interf_desc.bInterfaceProtocol == 0x01) {
+  if (is_adb_interface(interf_desc.bInterfaceClass, interf_desc.bInterfaceSubClass,
+                       interf_desc.bInterfaceProtocol)) {
+    if (interf_desc.bInterfaceProtocol == 0x01) {
       AdbEndpointInformation endpoint_info;
       // assuming zero is a valid bulk endpoint ID
       if (AdbGetEndpointInformation(handle->adb_interface, 0, &endpoint_info)) {