Crash-Test-Buddies / WiFi-Buddy

Wi-Fi Buddy - Library to handle Android Wi-Fi Direct
https://groups.google.com/forum/#!forum/wi-fi-direct
MIT License
48 stars 12 forks source link

Service discovery request added even if it's already added #146

Closed brendankirby closed 8 years ago

brendankirby commented 8 years ago

Below is everything that occurred from following these steps:

  1. Launch the app with Wi-Fi enabled
  2. Tap Discover Services
  3. Go back
  4. Tap Discover Services

The code prevents you from calling continuouslyDiscover if it's already discovery, but it doesn't prevent you from adding another service discovery request.

06-22 14:59:51.735 19459-19459/edu.rit.se.crashavoidance I/wfdMainFragment: Discover Services Button Pressed 06-22 14:59:51.751 19459-19459/edu.rit.se.crashavoidance I/wfd: Continuously Discover services called 06-22 14:59:51.751 19459-19459/edu.rit.se.crashavoidance I/wfd: Calling discover and submitting first discover task 06-22 14:59:51.751 19459-19459/edu.rit.se.crashavoidance I/wfd: Discover services called 06-22 14:59:51.752 19459-19459/edu.rit.se.crashavoidance I/wfd: Submitting discover task **06-22 14:59:51.773 19459-19459/edu.rit.se.crashavoidance I/wfd: Service discovery request added 06-22 14:59:51.779 19459-19459/edu.rit.se.crashavoidance I/wfd_: Service discovery initiated 06-22 14:59:53.831 19459-19459/edu.rit.se.crashavoidance I/wfd_MainFragment: Updating toggle switches 06-22 14:59:55.758 19459-19459/edu.rit.se.crashavoidance I/wfd_MainFragment: Discover Services Button Pressed 06-22 14:59:55.773 19459-19459/edu.rit.se.crashavoidance I/wfd: Continuously Discover services called 06-22 14:59:55.773 19459-19459/edu.rit.se.crashavoidance W/wfd: Services are still discovering, do not need to make this call 06-22 14:59:55.785 19459-19459/edu.rit.se.crashavoidance I/wfd_: Service discovery request added**

brendankirby commented 8 years ago

@celdridge91190 You're probably familiar with this

brendankirby commented 8 years ago

Fixed in PR 6 in wifi-direct-tester, closing....