- Implement a filter for devices that are repling multiple times.

- Fix the creation of the RemoteDevicesList


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@24990 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/kits/bluetooth/DiscoveryListener.cpp

@@ -5,9 +5,11 @@
  *
  */
 
+#include <bluetooth/DiscoveryAgent.h>
 #include <bluetooth/DiscoveryListener.h>
 #include <bluetooth/RemoteDevice.h>
 #include <bluetooth/DeviceClass.h>
+#include <bluetooth/bdaddrUtils.h>
 
 #include <bluetooth/HCI/btHCI_event.h>
 
@@ -69,21 +71,42 @@ DiscoveryListener::MessageReceived(BMessage* message)
         {
 			const struct inquiry_info* inquiryInfo;
 		 	ssize_t	size;
+			RemoteDevice* rd = NULL;
+			bool duplicatedFound = false;
 				
 			if (message->FindData("info", B_ANY_TYPE, 0, (const void**)&inquiryInfo, &size) == B_OK )
 			{
-	            RemoteDevice* rd = new RemoteDevice(inquiryInfo->bdaddr);
-	            //  DeviceClass(inquiryInfo->dev_class[0] | inquiryInfo->dev_class[1]<<8 | inquiryInfo->dev_class[2]<<16 )
-				//  fRemoteDevicesList.AddItem(rd);
-	            rd->SetLocalDeviceOwner(fLocalDevice);	      
-	            DeviceDiscovered( rd, DeviceClass(inquiryInfo->dev_class[0] | inquiryInfo->dev_class[1]<<8 | inquiryInfo->dev_class[2]<<16 ));
 
+			    // Skip duplicated replies
+			    for (int32 index = 0 ; index < fRemoteDevicesList.CountItems(); index++) {
+			    
+			    	bdaddr_t b1 = fRemoteDevicesList.ItemAt(index)->GetBluetoothAddress();
+			    				     	                          
+			        if (bdaddrUtils::Compare( (bdaddr_t*) &inquiryInfo->bdaddr, &b1 )) {
+
+						duplicatedFound = true;		
+			            break;			        
+			        }
+			        
+			    }
+
+				if (!duplicatedFound) {
+
+		            //  DeviceClass(inquiryInfo->dev_class[0] | inquiryInfo->dev_class[1]<<8 | inquiryInfo->dev_class[2]<<16 )
+       	            rd = new RemoteDevice(inquiryInfo->bdaddr);			        
+					fRemoteDevicesList.AddItem(rd);
+	    	        rd->SetLocalDeviceOwner(fLocalDevice);
+	        	    DeviceDiscovered( rd, DeviceClass(inquiryInfo->dev_class[0] | 
+	            	                                  inquiryInfo->dev_class[1]<<8 | 
+	                	                              inquiryInfo->dev_class[2]<<16 ));
+	            }    	                              
   			}
         }
         break;
 
         case BT_MSG_INQUIRY_STARTED:
             if (message->FindInt8("status", &status) == B_OK){
+                fRemoteDevicesList.MakeEmpty();
 	            InquiryStarted(status);
             }
 
@@ -116,7 +139,7 @@ DiscoveryListener::MessageReceived(BMessage* message)
 }
 
 
-DiscoveryListener::DiscoveryListener() : BLooper()
+DiscoveryListener::DiscoveryListener() : BLooper() , fRemoteDevicesList(BT_MAX_RESPONSES)
 {
 	// TODO: Make a better handling of the running not running state
 	Run();