-
Notifications
You must be signed in to change notification settings - Fork 398
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RFC: Bluetooth LE device info on Windows #309
Conversation
BLE Controllers example: Amazon Luna and Xbox One Model Model 1914. Before:
After:
|
This seems to work. Before (mainline git)
#309 output
|
2be8820
to
31bc524
Compare
Yes the latest git is good.
|
@DJm00n could you make a BLE fix only? BUS_TYPE isn't finished for all platforms, and even when it is done - let's have it in master for some time, before releaseing it. |
@DJm00n You may want to disable the following warning for VS.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe the warning is valid after all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a BLE device, so I can't really try this myself, but I've reviewed the implementation and it looks good, with regards to my last comments.
As soon as those are referred, I'll merge this.
04463c0
to
6009cf5
Compare
6009cf5
to
129daa3
Compare
129daa3
to
2433fc6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awsome, thanks!
This PR fixes issue with empty
manufacturer_string
/product_string
/serial_number
inhid_device_info
struct with Bluetooth LE devices on Windows systems.Fixes: #282